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
