[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|C
From: |
Maciej W. Rozycki |
Subject: |
Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D> |
Date: |
Mon, 20 Jun 2016 15:40:25 +0100 |
User-agent: |
Alpine 2.00 (DEB 1167 2008-08-23) |
On Mon, 20 Jun 2016, Aleksandar Markovic wrote:
> 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.
Well, if the matter does not affect MIPS emulation, then why do you think
there's a need to mention it in the first place?
I don't think stating what other emulation targets do in a commit
description for a platform feature adds any value, but I think it bears
the risk to confuse people examining code history. Your description has
to be meaningful for someone reading it say 10 years on from now who does
not necessarily know the context nor has any of us available to ask.
Please consider this.
If you do think this mention adds value, then please make it unambiguous
in the commit description that the third conversion variant applies to
non-MIPS targets only. It wasn't to me at all or I wouldn't have spoken
out.
Thank you.
Maciej
- Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>, Maciej W. Rozycki, 2016/06/07
- Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>, Aleksandar Markovic, 2016/06/10
- Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>, Maciej W. Rozycki, 2016/06/10
- Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>, Aleksandar Markovic, 2016/06/10
- Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>, Maciej W. Rozycki, 2016/06/10
- Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>, Leon Alrae, 2016/06/14
- Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>, Maciej W. Rozycki, 2016/06/14
- Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>, Aleksandar Markovic, 2016/06/20
- Re: [Qemu-ppc] [PATCH v6 8/9] target-mips: Add nan2008 flavor of <CEIL|CVT|FLOOR|ROUND|TRUNC>.<L|W>.<S|D>,
Maciej W. Rozycki <=