qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Alexander Graf <alex@csgraf.de>
To: Thiemo Seufer <ths@networkno.de>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 2/5] Add a PowerPC branch detector
Date: Thu, 17 Jan 2008 15:40:25 +0100	[thread overview]
Message-ID: <478F68D9.7030808@csgraf.de> (raw)
In-Reply-To: <20080117143813.GB28842@networkno.de>

Thiemo Seufer wrote:
> Johannes Schindelin wrote:
>   
>> Hi,
>>
>> On Thu, 17 Jan 2008, Thiemo Seufer wrote:
>>
>>     
>>> Alexander Graf wrote:
>>> [snip]
>>>       
>>>>> Is check_ops.sh not enough for debugging micro-ops?
>>>>>
>>>>>           
>>>> Basically it should be. PowerPC branching can be (regex) b..rl. Honestly
>>>> I did not know about this script though and as it was not in the
>>>> makefile, it did not tell me that something wrong was going on. This
>>>> check costs near no time and has to be passed in order to build
>>>> successfully.
>>>>
>>>> So either check_ops should be fixed (not only brl) and put into the
>>>> Makefile.target or a check like this is good to have.
>>>>         
>>> Fixing check_ops is IMO preferable, OTOH I hope the whole problem goes
>>> away with the new code generator which is in the works.
>>>       
>> The code generated by that new code generator is substantially slower than 
>> what we have right now, so I am not so enthusiastic.
>>     
>
> I meant not qops but something Fabrice is working on. Which apparently has
> about the same speed.
>
>   

Either way I wouldn't mind having a working gcc4 version for the time
being, including checks that it doesn't break too easily again.

Alex

  reply	other threads:[~2008-01-17 14:40 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-17  6:42 [Qemu-devel] [PATCH 2/5] Add a PowerPC branch detector Alexander Graf
2008-01-17 13:22 ` Thiemo Seufer
2008-01-17  9:48   ` Alexander Graf
2008-01-17 14:07     ` Thiemo Seufer
2008-01-17 14:16       ` Johannes Schindelin
2008-01-17 14:38         ` Thiemo Seufer
2008-01-17 14:40           ` Alexander Graf [this message]
2008-01-17 14:56           ` Andreas Färber

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=478F68D9.7030808@csgraf.de \
    --to=alex@csgraf.de \
    --cc=qemu-devel@nongnu.org \
    --cc=ths@networkno.de \
    /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).