qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL


From: Aleksandar Markovic
Subject: Re: [Qemu-devel] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>
Date: Mon, 20 Jun 2016 10:20:33 +0000

This patch is about NaN-2008 flavor of Mips instructions CEIL, CVT, FLOOR, 
ROUND, TRUNC only (its title is "Add nan2008 flavor...").

Legacy-NaN flavors of the same Mips instructions already operate correctly, and 
there is nothing to be fixed.

The commit message is not description of another bug, it explains the context 
of the change, and the content of added code segments.

________________________________________
From: Maciej Rozycki
Sent: Tuesday, June 14, 2016 8:30 AM
To: Leon Alrae
Cc: Aleksandar Markovic; Aleksandar Markovic; address@hidden; address@hidden; 
address@hidden; address@hidden; address@hidden; address@hidden; address@hidden; 
address@hidden; address@hidden; address@hidden; Petar Jovanovic; 
address@hidden; Miodrag Dinic; address@hidden; address@hidden; address@hidden; 
Aurelien Jarno; address@hidden
Subject: Re: [PATCH v6 8/9] target-mips: Add nan2008 flavor of 
<CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>

On Tue, 14 Jun 2016, Leon Alrae wrote:

> > 1. A bug fix for SoftFloat legacy-NaN (original) MIPS support, which has
> >    been there probably since forever (i.e. since the MIPS target was added
> >    to QEMU).
>
> I've just done another round of review and as far as I can tell these
> patches don't modify the legacy-NaN MIPS behaviour. I believe Aleksandar
> was referring to new functionality (i.e. 2008 NaN) only.

 Well, if there's a bug, then it needs to be fixed rather than documented.

  Maciej



reply via email to

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