[cfe-dev] build failure: AArch64BaseInfo.h has questionable enum values

NAKAMURA Takumi geek4civic at gmail.com
Tue Nov 19 18:03:40 PST 2013


You should take a look in you r192361, FYI.

2013/11/20 Hao Liu <haoliuts at gmail.com>:
> Hi,
>
> Eric, this is added by me.
> How about changing them :
>     B,
>     H,
>     S,
>     D
>
> Thanks,
> -Hao
>
>
> 2013/11/20 Eric Niebler <eniebler at boost.org>
>>
>> lib/Target/AArch64/Utils/AArch64BaseInfo.h has the following enum:
>>
>> namespace A64Layout {
>>     enum VectorLayout {
>>         Invalid = -1,
>> /*...*
>>         // Bare layout for the 128-bit vector
>>         // (only show ".b", ".h", ".s", ".d" without vector number)
>>         _B,
>>         _H,
>>         _S,
>>         _D
>>     };
>> }
>>
>> My build on cygwin fails because _B and _S are platform macros. Besides,
>> in C++ identifiers with a leading underscore-capital are reserved. I
>> suggest changing these four.
>>
>> --
>> Eric Niebler
>> Boost.org
>> http://www.boost.org
>> _______________________________________________
>> cfe-dev mailing list
>> cfe-dev at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
>
>
>
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
>



More information about the cfe-dev mailing list