[llvm-commits] [llvm-gcc-4.2] r63322 - /llvm-gcc-4.2/trunk/gcc/config/i386/i386.c

[Chris Lattner]

On Jan 29, 2009, at 10:22 AM, Dale Johannesen wrote:

>
> On Jan 29, 2009, at 10:16 AMPST, Chris Lattner wrote:
>> URL: http://llvm.org/viewvc/llvm-project?rev=63322&view=rev
>> Log:
>> Default -fno-math-errno to on with all i386 targets.  This is after
>> discussion
>> with Anton and Duncan.  -fmath-errno still works to enable it.
>
> This is a violation of the C89 standard and can break programs that
> are correct under that standard.  It may be the right thing to do
> anyway, but the justification should mention that you've taken this
> into account.

I intend to fully document this in the release notes for 2.5.  Since  
the program is already non-portable (at least to macs) and C99 doesn't  
require this, I think it is ok.

Thanks Dale!

-Chris