From: Richard Henderson <rth@twiddle.net>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: aurelien@aurel32.net, qemu-devel@nongnu.org, anthony@codemonkey.ws
Subject: Re: [Qemu-devel] [PULL 5/7] tcg: Add mmu helpers that take a return address argument
Date: Mon, 26 Aug 2013 15:34:15 -0700 [thread overview]
Message-ID: <521BD7E7.9090107@twiddle.net> (raw)
In-Reply-To: <521BD62A.50203@redhat.com>
On 08/26/2013 03:26 PM, Paolo Bonzini wrote:
> Something that can be done on top of this patch: what about moving the
> "-1" to helper_ret_*? It is common to pretty much all the targets
> (except ARM has -2), and it would allow some simplifications.
I suppose so, yes.
> li rN, retaddr
> mtlr rN
> b st_trampoline[i]
>
> sequence instead of one of
>
> li rN, retaddr
> mtlr rN
> bl st_trampoline[i]
> b retaddr
This sort of thing is very difficult to evaluate, because of the
cpu's return address prediction stack. I have so far avoided it.
The only cpus that I believe can make good use of tail calls into
the memory helpers are those with predicated stores and calls, i.e.
arm and ia64.
r~
next prev parent reply other threads:[~2013-08-26 22:34 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-26 21:00 [Qemu-devel] [PULL 0/7] Improve tcg ldst optimization Richard Henderson
2013-08-26 21:00 ` [Qemu-devel] [PULL 1/7] tcg: Tidy generated code for tcg_outN Richard Henderson
2013-08-26 21:00 ` [Qemu-devel] [PULL 2/7] tcg-i386: Add and use tcg_out64 Richard Henderson
2013-08-26 21:00 ` [Qemu-devel] [PULL 3/7] tcg-i386: Try pc-relative lea for constant formation Richard Henderson
2013-08-26 21:00 ` [Qemu-devel] [PULL 4/7] tcg-i386: Tidy qemu_ld/st slow path Richard Henderson
2013-08-26 21:00 ` [Qemu-devel] [PULL 5/7] tcg: Add mmu helpers that take a return address argument Richard Henderson
2013-08-26 22:26 ` Paolo Bonzini
2013-08-26 22:34 ` Richard Henderson [this message]
2013-08-27 10:46 ` Aurelien Jarno
2013-08-27 14:53 ` Richard Henderson
2013-08-27 15:43 ` Aurelien Jarno
2013-08-27 15:53 ` Richard Henderson
2013-08-26 23:26 ` Peter Maydell
2013-08-27 10:47 ` Aurelien Jarno
2013-08-26 21:00 ` [Qemu-devel] [PULL 6/7] tcg: Tidy softmmu_template.h Richard Henderson
2013-08-26 21:00 ` [Qemu-devel] [PULL 7/7] tcg-i386: Use new return-argument ld/st helpers Richard Henderson
2013-08-28 22:55 ` Richard W.M. Jones
2013-08-29 15:21 ` [Qemu-devel] [PATCH] tcg-i386: Remove abort from GETPC_LDST Richard Henderson
2013-08-29 15:28 ` Paolo Bonzini
2013-08-29 15:32 ` Richard Henderson
2013-08-29 15:50 ` Richard W.M. Jones
2013-08-29 19:13 ` Aurelien Jarno
2013-08-27 21:30 ` [Qemu-devel] [PULL 0/7] Improve tcg ldst optimization Aurelien Jarno
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=521BD7E7.9090107@twiddle.net \
--to=rth@twiddle.net \
--cc=anthony@codemonkey.ws \
--cc=aurelien@aurel32.net \
--cc=pbonzini@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.