From: Laurent Desnogues <laurent.desnogues@gmail.com>
To: Juha.Riihimaki@nokia.com
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v2 09/10] target-arm: optimize neon vld/vst ops
Date: Mon, 26 Oct 2009 10:11:07 +0100 [thread overview]
Message-ID: <761ea48b0910260211k12d199c5l794eb0f9d45e3b77@mail.gmail.com> (raw)
In-Reply-To: <4F9750C3-A196-4518-8021-D02BFE645A4F@nokia.com>
On Mon, Oct 26, 2009 at 8:46 AM, <Juha.Riihimaki@nokia.com> wrote:
>
> On Oct 25, 2009, at 16:01, ext Laurent Desnogues wrote:
>
>> I don't really like the idea of having tcg_qemu_ld/st not factored
>> in some place, as it makes memory access tracing extensions
>> more intrusive.
>>
>> This brings us back to the problem having function freeing tmps.
>> In that case, you could perhaps create a gen_st16_dont_free
>> function as a temporary workaround?
>
> I could, but it is getting ugly =/ How about if I create another patch
> that moves the temporary variable (de)allocation outside the gen_ldx/
> gen_stx functions and then refactor this patch on top of that?
I'd personally like this, but I guess you'd better wait for some inputs
from other QEMU dev's before doing it.
Laurent
next prev parent reply other threads:[~2009-10-26 9:11 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-24 12:18 [Qemu-devel] [PATCH v2 00/10] target-arm: miscellaneous fixes juha.riihimaki
2009-10-24 12:19 ` [Qemu-devel] [PATCH v2 01/10] target-arm: fix neon vshrn/vrshrn ops juha.riihimaki
2009-10-25 10:58 ` Laurent Desnogues
2009-10-24 12:19 ` [Qemu-devel] [PATCH v2 02/10] target-arm: add support for neon vld1.64/vst1.64 instructions juha.riihimaki
2009-10-25 11:11 ` Laurent Desnogues
2009-10-24 12:19 ` [Qemu-devel] [PATCH v2 03/10] target-arm: allow modifying vfp fpexc en bit only juha.riihimaki
2009-10-25 12:23 ` Laurent Desnogues
2009-10-26 7:32 ` Juha.Riihimaki
2009-10-26 9:08 ` Laurent Desnogues
2009-10-24 12:19 ` [Qemu-devel] [PATCH v2 04/10] target-arm: optimize vfp load/store multiple ops juha.riihimaki
2009-10-24 17:36 ` Laurent Desnogues
2009-10-27 8:42 ` Aurelien Jarno
2009-10-24 12:19 ` [Qemu-devel] [PATCH v2 05/10] target-arm: optimize arm " juha.riihimaki
2009-10-27 8:39 ` Aurelien Jarno
2009-10-27 8:48 ` Juha.Riihimaki
2009-10-27 9:00 ` Aurelien Jarno
2009-10-27 9:05 ` Juha.Riihimaki
2009-10-24 12:19 ` [Qemu-devel] [PATCH v2 06/10] target-arm: fix neon vsri, vshl and vsli ops juha.riihimaki
2009-10-24 17:44 ` Laurent Desnogues
2009-10-24 12:19 ` [Qemu-devel] [PATCH v2 07/10] target-arm: optimize thumb2 load/store multiple ops juha.riihimaki
2009-10-24 17:32 ` Laurent Desnogues
2009-10-24 12:19 ` [Qemu-devel] [PATCH v2 08/10] target-arm: optimize thumb push/pop ops juha.riihimaki
2009-10-24 17:34 ` Laurent Desnogues
2009-10-24 12:19 ` [Qemu-devel] [PATCH v2 09/10] target-arm: optimize neon vld/vst ops juha.riihimaki
2009-10-25 14:01 ` Laurent Desnogues
2009-10-26 7:46 ` Juha.Riihimaki
2009-10-26 9:11 ` Laurent Desnogues [this message]
2009-10-26 21:05 ` Aurelien Jarno
2009-10-29 13:45 ` Juha.Riihimaki
2009-10-29 13:52 ` Laurent Desnogues
2009-10-24 12:19 ` [Qemu-devel] [PATCH v2 10/10] target-arm: fix neon shift helper functions juha.riihimaki
2009-10-25 12:16 ` Laurent Desnogues
2009-10-25 19:17 ` [Qemu-devel] [PATCH v2 00/10] target-arm: miscellaneous fixes 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=761ea48b0910260211k12d199c5l794eb0f9d45e3b77@mail.gmail.com \
--to=laurent.desnogues@gmail.com \
--cc=Juha.Riihimaki@nokia.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).