qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Laurent Desnogues" <laurent.desnogues@gmail.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] ppc: Convert op_440_dlmzb to TCG
Date: Sun, 30 Nov 2008 19:45:13 +0100	[thread overview]
Message-ID: <761ea48b0811301045g563be9ddg357d6115cd8ea26@mail.gmail.com> (raw)
In-Reply-To: <0994E4E8-B10A-4449-8639-5877689A7D28@web.de>

On Sun, Nov 30, 2008 at 7:03 PM, Andreas Färber <andreas.faerber@web.de> wrote:
>
> So, if I encounter a dyngen op that calls a helper, then in general I should
> convert it to TCG code calling the helper?

I guess it depends, but in most cases probably.  If an instruction is
not used often and its implementation in the old helper is rather
long or uses branches (if/else, loops), you should keep it as a
helper.

> My next problem is op_440_tlbre, which does call a helper, but with a PARAM1
> argument. Any hint how to translate that?

I am not familiar at all with the old generator, so take what I say with
a grain of salt :-)

PARAM1 probably was a parameter passed at runtime.  Now with
the way helper calls are generated, you explicitly pass parameters.
Your best bet is to look at how Aurélien reimplemented dlmzb or
any other complex helpers since he started the conversion to TCG.


Laurent

  reply	other threads:[~2008-11-30 18:45 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-29 14:35 [Qemu-devel] [PATCH] ppc: Convert op_440_dlmzb to TCG Andreas Färber
2008-11-30 16:31 ` Aurélien Jarno
2008-11-30 17:25   ` Andreas Färber
2008-11-30 17:40     ` Laurent Desnogues
2008-11-30 18:03       ` Andreas Färber
2008-11-30 18:45         ` Laurent Desnogues [this message]
2008-11-30 17:27   ` Aurélien 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=761ea48b0811301045g563be9ddg357d6115cd8ea26@mail.gmail.com \
    --to=laurent.desnogues@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).