[llvm-commits] [llvm-gcc-4.2] r103366 - in /llvm-gcc-4.2/trunk/gcc: config/darwin-c.c config/t-darwin testsuite/gcc.apple/opt-pragma-1.c testsuite/gcc.apple/opt-pragma-2.c testsuite/gcc.apple/opt-pragma-3.c testsuite/gcc.apple/opt-pragma-4.c testsuite/gcc.apple/opt-pragma-5.c testsuite/gcc.apple/opt-pragma-6.c testsuite/gcc.apple/opt-pragma-7.c

[Dale Johannesen]

On May 9, 2010, at 10:37 AMPDT, Chris Lattner wrote:
> On May 9, 2010, at 9:22 AM, Dale Johannesen wrote:
> On May 8, 2010, at 4:43 PM, Eric Christopher wrote:
>> 
>>> Author: echristo
>>> Date: Sat May  8 18:43:20 2010
>>> New Revision: 103366
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=103366&view=rev
>>> Log:
>>> Disable optimization pragmas.
>>> 
>>> Fixes rdar://7820636
>> 
>> There are people using these.  You will certainly get complaints.
> 
> They don't work in llvm-gcc and clang doesn't support them.  It seems no harm.

It may be true that we don't need the functionality if nobody has noticed it doesn't work.  But now they silently don't work, and the change gives a warning.  This will break projects that build with -Werror.