Re: [boost] [shared_ptr] Clang with Microsoft CodeGen in VS 2015 Update1

Subject: Re: [boost] [shared_ptr] Clang with Microsoft CodeGen in VS 2015 Update1
From: Marcel Raad (raad_at_[hidden])
Date: 2015-12-09 09:21:08

Beman Dawes wrote
> On Tue, Dec 8, 2015 at 10:55 AM, Peter Dimov <

> lists@

> > wrote:
> 
>> Beman Dawes wrote:
>>
>> So it looks like the clang/c2 toolset doesn't support atomics yet. The
>>> __c2__ and __c2_version__ macros allow detection of the clang/c2
>>> toolset,
>>> could a workaround be added to shared_ptr?
>>>
>>
>> If you change
>>
>> #if defined( __clang__ ) && defined( __has_extension )
>> # if __has_extension( __c_atomic__ )
>> #   define BOOST_SP_HAS_CLANG_C11_ATOMICS
>> # endif
>> #endif
>>
>> in sp_counted_base.hpp to
>>
>> #if defined( __clang__ ) && defined( __has_extension ) && !defined(
>> __c2__
>> )
>> # if __has_extension( __c_atomic__ )
>> #   define BOOST_SP_HAS_CLANG_C11_ATOMICS
>> # endif
>> #endif
>>
>> does it work then?
>>
>>
> I also had to do the same for two other cases, but in the end
> sp_counted_base_w32.hpp worked and the entire filesystem dll built OK.

The same problem occurs with clang-cl, which defines _MSC_VER instead of
__GNUC__. Could that be fixed too please?

clang/c2 defines __GNUC__ but doesn't support GCC inline assembly, so
sp_counted_base_gcc_x86.hpp also has to be excluded for __c2__.

--
View this message in context: http://boost.2283326.n4.nabble.com/shared-ptr-Clang-with-Microsoft-CodeGen-in-VS-2015-Update-1-tp4681900p4681954.html
Sent from the Boost - Dev mailing list archive at Nabble.com.