All of lore.kernel.org
 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 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.