qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] Re: dyngen-exec.h cleanup


From: Jan Kiszka
Subject: [Qemu-devel] Re: dyngen-exec.h cleanup
Date: Sun, 08 Mar 2009 17:39:24 +0100
User-agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666

Laurent Desnogues wrote:
> On Sun, Mar 8, 2009 at 5:00 PM, Jan Kiszka <address@hidden> wrote:
>> Laurent Desnogues wrote:
> [...]
>>> Yes, and I did the work for ARM.  However when considering the
>>> removal of AREG0, and after looking at generated code, I came
>>> to the perhaps premature conclusion that removing it would not
>>> bring me any speedup (at least for a not so register starved
>>> target as x86_64).
>> I don't think we are looking for speedup here, just for cleanup. Status
>> quo regarding performance after a conversion would be more than fine IMHO.
> 
> I was ambiguous, I meant I expect slowdowns...  But given how many
> times I was wrong in the past when I thought some tricks would
> speedup things, I should probably not trust my a priori.
> 
>>> Well dyngen-exec.h is long gone in my sources even though
>>> AREG0 is still used.  I would have to backtrack my changes to
>>> see how I arrived to that, but for sure the first thing to do is to
>>> remove cpu_T from ARM target.
>> Yes, please share your wisdom!
> 
> Sorry if I sounded arrogant, I just wanted to highlight the fact one
> can achieve your aim.  As far as sharing goes, that's another
> story.

No, no, you didn't sound arrogant. I just forgot your boundary conditions.

But it's good to know that there might be a way to overcome
dyngen-exec.h without dropping AREGs altogether in case performance
requires this.

Jan

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]