[llvm] r194365 - Don't universally enable initialiser lists on GCC. Thanks for catching this Chandler

Eric Christopher echristo at gmail.com
Mon Nov 11 17:27:00 PST 2013


Enh, nevermind, the existing code is fine and we're moving to C++11 soon anyhow.

-eric

On Mon, Nov 11, 2013 at 10:12 AM, Eric Christopher <echristo at gmail.com> wrote:
> Even better would be to piggy back this on top of the existing
> "compile in C++11" mode and you can then also check whether or not the
> feature exists via cmake or autoconf.
>
> -eric
>
> On Sun, Nov 10, 2013 at 9:14 PM, Pete Cooper <peter_cooper at apple.com> wrote:
>> Author: pete
>> Date: Sun Nov 10 23:14:42 2013
>> New Revision: 194365
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=194365&view=rev
>> Log:
>> Don't universally enable initialiser lists on GCC.  Thanks for catching this Chandler
>>
>> Modified:
>>     llvm/trunk/include/llvm/Support/Compiler.h
>>
>> Modified: llvm/trunk/include/llvm/Support/Compiler.h
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/Compiler.h?rev=194365&r1=194364&r2=194365&view=diff
>> ==============================================================================
>> --- llvm/trunk/include/llvm/Support/Compiler.h (original)
>> +++ llvm/trunk/include/llvm/Support/Compiler.h Sun Nov 10 23:14:42 2013
>> @@ -405,8 +405,7 @@
>>
>>  /// \brief Does the compiler support generalized initializers (using braced
>>  /// lists and std::initializer_list).
>> -#if (__has_feature(cxx_generalized_initializers)   \
>> -|| defined(__GXX_EXPERIMENTAL_CXX0X__))
>> +#if __has_feature(cxx_generalized_initializers)
>>  #define LLVM_HAS_INITIALIZER_LISTS 1
>>  #else
>>  #define LLVM_HAS_INITIALIZER_LISTS 0
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list