[llvm-commits] [llvm-gcc-4.2] r100565 - /llvm-gcc-4.2/trunk/gcc/c-parser.c

[Bob Wilson]

On Apr 6, 2010, at 3:05 PM, Dale Johannesen wrote:

> 
> On Apr 6, 2010, at 3:03 PMPDT, Eli Friedman wrote:
> 
>> On Tue, Apr 6, 2010 at 2:56 PM, Bob Wilson <bob.wilson at apple.com> wrote:
>>> Author: bwilson
>>> Date: Tue Apr  6 16:56:05 2010
>>> New Revision: 100565
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=100565&view=rev
>>> Log:
>>> Disable the "ISO C forbids braced-groups within expressions" pedantic
>>> warning.  This is needed for llvm's implementation of the ARM NEON intrinsics,
>>> which use macros with statements expressions instead of inline functions.
>>> This warning made is impossible to compile NEON intrinsics with "-pedantic
>>> -Werror".  Radar 7833512.
>> 
>> Umm, this isn't really good... would it be possible to change the
>> header in question to use __extension__ to avoid the warnings?
> 
> And does clang have the same issue?

Clang has not yet implemented the NEON intrinsics, and when it does it should not be hard to avoid the problems that llvm-gcc has.  llvm-gcc inherited a bunch of issues from the original gcc implementation of the NEON intrinsics, which we tried to reuse.  In retrospect we might have been better off starting from scratch....