From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from [140.186.70.92] (port=48820 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1PavTI-0003Vl-Ay
	for qemu-devel@nongnu.org; Thu, 06 Jan 2011 14:26:25 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <sdb@zubnet.me.uk>) id 1PavTH-0000fg-4a
	for qemu-devel@nongnu.org; Thu, 06 Jan 2011 14:26:04 -0500
Received: from b.painless.aaisp.net.uk ([81.187.30.52]:52700)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <sdb@zubnet.me.uk>) id 1PavTG-0000fK-NU
	for qemu-devel@nongnu.org; Thu, 06 Jan 2011 14:26:03 -0500
Date: Thu, 6 Jan 2011 19:25:58 +0000
From: Stuart Brady <sdb@zubnet.me.uk>
Subject: Re: [Qemu-devel] [PATCH 1/6] softfloat: remove HPPA specific code
Message-ID: <20110106192558.GA2118@zubnet.me.uk>
References: <151C200F-E9B1-4BDC-B234-C0B03D3086D2@web.de>
	<20110104200711.GC3615@hall.aurel32.net>
	<10FC1308-1CC8-41DF-A48C-AD104BCE6E11@web.de>
	<20110104235604.GD3615@hall.aurel32.net>
	<072B9AB0-7B44-4A79-80FE-A3D95914AAD4@web.de>
	<20110105102106.GD15256@volta.aurel32.net>
	<20110105231306.GA20254@zubnet.me.uk>
	<AANLkTimbFzTO4RR=JYH_xFb-DHa9t+BrWE=RV8WKPfYB@mail.gmail.com>
	<20110106181320.GA2180@zubnet.me.uk>
	<AANLkTikuZ3tpcs_OVG_jocwTW=mdQaz=Ez01=LZwHixM@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
In-Reply-To: <AANLkTikuZ3tpcs_OVG_jocwTW=mdQaz=Ez01=LZwHixM@mail.gmail.com>
Content-Transfer-Encoding: quoted-printable
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.nongnu.org/archive/html/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: Peter Maydell <peter.maydell@linaro.org>
Cc: Andreas =?iso-8859-1?Q?F=E4rber?= <andreas.faerber@web.de>, Riku Voipio <riku.voipio@iki.fi>, QEMU Developers <qemu-devel@nongnu.org>, Aurelien Jarno <aurelien@aurel32.net>

On Thu, Jan 06, 2011 at 06:43:28PM +0000, Peter Maydell wrote:
> On 6 January 2011 18:13, Stuart Brady <sdb@zubnet.me.uk> wrote:
> > On Thu, Jan 06, 2011 at 08:58:17AM +0000, Peter Maydell wrote:
> >> On 5 January 2011 23:13, Stuart Brady <sdb@zubnet.me.uk> wrote:
> >> > I do have a few concerns regarding SoftFloat, though:
> >> >
> >> > =A0 FIXMEs should be left in the code (or a document maintained on=
 the
> >> > =A0 Wiki) to keep track of which architectures have been considere=
d
> >> > =A0 (which I believe are x86, arm, mips, ppc) and which ones haven=
't.
> >> > =A0 This is in reference to one particular FIXME that was removed,
> >> > =A0 but perhaps shouldn't have been.
[...]
> > =A0 /* FIXME: Flush-To-Zero only effects results. =A0Denormal inputs =
should
> > =A0 =A0 =A0also be flushed to zero. =A0*/
>=20
> The point of that FIXME is that it is saying "softfloat doesn't impleme=
nt
> the feature of flushing denormal inputs to zero". The patch implements
> that feature in softfloat. Therefore the FIXME should be removed,
> because it has been fixed :-)

Agreed, although note that I never insisted that the FIXME be kept in
this instance -- I was just concerned that this might be forgotten
for other targets.

If we accept that those targets are likely to be buggy, and that this
is an issue for the maintainers of those targets, that sounds fine,
but I just thought I'd ask.

Cheers,
--=20
Stuart Brady