If libcxx doesn't build with Visual Studio, then Chandler's patch to enable it 
should be changed to make it conditional on not using VS.  Chandler, can that 
be done?

On Oct 1, 2013, at 8:56 AM, jahanian <[email protected]> wrote:

> We can disable build of libcxx on our VS buildbots. But, we would like to 
> hear better alternatives/suggestions, if any.
> 
> - Thanks, Fariborz
> 
> 
> On Sep 30, 2013, at 9:03 AM, jahanian <[email protected]> wrote:
> 
>> HI Chandler,
>> This patch 
>> Enable libcxx as part of the top level CMake build when it is checked
>> out in projects. This appears to be working on my system, and I will be
>> watching build bots to see if there are any issues on other platforms.
>> 
>> Changed files
>> 
>> src/projects/CMakeLists.txt
>> 
>> causes all libcxx compiles to fail on VS buildbot. AN example of diagnostics 
>> would be:
>> 
>>  hash.cpp
>> C:/public/zorg/buildbot/smooshlab/slave-0.8/.../src/projects/libcxx/include\type_traits(927):
>>  error C2681: 'add_rvalue_reference<_Tp*>::type' : invalid expression type 
>> for dynamic_cast 
>> [C:\public\zorg\buildbot\smooshlab\slave-0.8\...\clang-build\210328\projects\libcxx\lib\cxx.vcxproj]
>> C:/public/zorg/buildbot/smooshlab/slave-0.8/.../src/projects/libcxx/include\type_traits(927):
>>  error C2681: 'add_rvalue_reference<_Tp*>::type' : invalid expression type 
>> for dynamic_cast 
>> [C:\public\zorg\buildbot\smooshlab\slave-0.8\...\clang-build\210328\projects\libcxx\lib\cxx.vcxproj]
>> C:/public/zorg/buildbot/smooshlab/slave-0.8/.../src/projects/libcxx/include\type_traits(927):
>>  error C2681: 'add_rvalue_reference<_Tp*>::type' : invalid expression type 
>> for dynamic_cast 
>> [C:\public\zorg\buildbot\smooshlab\slave-0.8\...\clang-build\210328\projects\libcxx\l
>> ib\cxx.vcxproj]C:/public/zorg/buildbot/smooshlab/slave-0.8/.../src/projects/libcxx/include\type_traits(1404):
>>  error C2516: 'std::common_type<_Tp>::type' : is not a legal base class 
>> [C:\public\zorg\buildbot\smooshlab\slave-0.8\...\clang-build\210328\projects\libcxx\lib\cxx.vcxproj]
>>           with
>>           [
>>               _Tp=
>>           ]
>>           
>> C:/public/zorg/buildbot/smooshlab/slave-0.8/.../src/projects/libcxx/include\type_traits(1328)
>>  : see declaration of 'std::common_type<_Tp>::type'
>>           with
>>           [
>>               _Tp=
>>           ]
>> 
>> - Fariborz
>> 
>> 
>> _______________________________________________
>> cfe-commits mailing list
>> [email protected]
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> 
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to