From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1JFVs4-0006by-QD
	for qemu-devel@nongnu.org; Thu, 17 Jan 2008 09:37:32 -0500
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1JFVs1-0006bC-17
	for qemu-devel@nongnu.org; Thu, 17 Jan 2008 09:37:32 -0500
Received: from [199.232.76.173] (helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1JFVs0-0006b9-Og
	for qemu-devel@nongnu.org; Thu, 17 Jan 2008 09:37:28 -0500
Received: from relay01.mx.bawue.net ([193.7.176.67])
	by monty-python.gnu.org with esmtps
	(TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60)
	(envelope-from <ths@networkno.de>) id 1JFVs0-0005tH-9A
	for qemu-devel@nongnu.org; Thu, 17 Jan 2008 09:37:28 -0500
Date: Thu, 17 Jan 2008 14:38:13 +0000
From: Thiemo Seufer <ths@networkno.de>
Subject: Re: [Qemu-devel] [PATCH 2/5] Add a PowerPC branch detector
Message-ID: <20080117143813.GB28842@networkno.de>
References: <478EF8E3.2050503@csgraf.de> <20080117132253.GE9767@networkno.de>
	<478F2451.4090301@csgraf.de> <20080117140759.GA28842@networkno.de>
	<alpine.LSU.1.00.0801171415540.17650@racer.site>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <alpine.LSU.1.00.0801171415540.17650@racer.site>
Reply-To: qemu-devel@nongnu.org
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: Alexander Graf <alex@csgraf.de>, qemu-devel@nongnu.org

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.


Thiemo