public inbox for [email protected]  

help / color / mirror / Atom feed

From: Andreas Karlsson <[email protected]>
To: Xing Guo <[email protected]>
To: [email protected]
Subject: Re: JIT: Remove some unnecessary instructions.
Date: Fri, 30 Aug 2024 14:50:51 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <CACpMh+C=g13WdvzLRSponsVWGgxwDSMzQWM4Gz0heOyaA0-N6g@mail.gmail.com>
References: <CACpMh+C=g13WdvzLRSponsVWGgxwDSMzQWM4Gz0heOyaA0-N6g@mail.gmail.com>

On 8/30/24 5:55 AM, Xing Guo wrote:
> I find there are some unnecessary load/store instructions being
> emitted by the JIT compiler.

Well spotted! All of these are obvious dead instructions and while LLVM 
might be able to optimize them away there is no reason to create extra 
work for the optimizer.

The patch looks good, applies and the tests passes.

Andreas

view thread (4+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: JIT: Remove some unnecessary instructions.
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox