qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Richard Henderson <rth@twiddle.net>
To: Artyom Tarasenko <atar4qemu@gmail.com>
Cc: qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PULL 5/8] tcg: Put opcodes in a linked list
Date: Mon, 17 Aug 2015 08:44:08 -0700	[thread overview]
Message-ID: <55D20148.9060303@twiddle.net> (raw)
In-Reply-To: <CACXAS8A2_DdvrKC3oH5fYxHymxM_RiYGycy9qSp-Q1QOp_5X0g@mail.gmail.com>

On 08/17/2015 04:35 AM, Artyom Tarasenko wrote:
> Hi Richard,
> 
> this patch seems to break a build when USE_LIVENESS_ANALYSIS is undefined.

I suppose that's possible.  If so, it must be a trivial error somewhere.
Why in the world would you want !USE_LIVENESS_ANALYSIS?

Are you simply trying to see what sort of performance impact it has?  I can
tell you it's fairly important for generating x86 code.  The two operand nature
of the ISA means that if we don't know that the source operand is dead, we
often have to introduce an extra move to preserve it.


r~

  reply	other threads:[~2015-08-17 15:44 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-13  5:43 [Qemu-devel] [PULL 0/8] Linked list for tcg ops Richard Henderson
2015-02-13  5:43 ` [Qemu-devel] [PULL 1/8] tcg: Move some opcode generation functions out of line Richard Henderson
2015-02-13  5:43 ` [Qemu-devel] [PULL 2/8] tcg: Reduce ifdefs in tcg-op.c Richard Henderson
2015-02-13  5:43 ` [Qemu-devel] [PULL 3/8] tcg: Move emit of INDEX_op_end into gen_tb_end Richard Henderson
2015-02-13  5:43 ` [Qemu-devel] [PULL 4/8] tcg: Introduce tcg_op_buf_count and tcg_op_buf_full Richard Henderson
2015-02-13  5:43 ` [Qemu-devel] [PULL 5/8] tcg: Put opcodes in a linked list Richard Henderson
2015-08-17 11:35   ` Artyom Tarasenko
2015-08-17 15:44     ` Richard Henderson [this message]
2015-08-17 16:15       ` Artyom Tarasenko
2015-02-13  5:43 ` [Qemu-devel] [PULL 6/8] tcg: Remove opcodes instead of noping them out Richard Henderson
2015-02-13  5:43 ` [Qemu-devel] [PULL 7/8] tcg: Implement insert_op_before Richard Henderson
2015-02-13  5:43 ` [Qemu-devel] [PULL 8/8] tcg: Remove unused opcodes Richard Henderson
2015-02-13 12:49 ` [Qemu-devel] [PULL 0/8] Linked list for tcg ops Peter Maydell

Reply instructions:

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

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

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55D20148.9060303@twiddle.net \
    --to=rth@twiddle.net \
    --cc=atar4qemu@gmail.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).