BOOST_INTERLOCKED_EXCHANGE_ADD doesn't work with clang(-cl) about winapi HOT 6 CLOSED

I don't have clang to test. @pdimov, would you like to take a look?

from winapi.

The smart_ptr-specific version of interlocked.hpp is at https://github.com/boostorg/smart_ptr/blob/develop/include/boost/smart_ptr/detail/sp_interlocked.hpp.

I don't remember all the intricacies of this - it's a pain of getting it to work on all the various configurations, each with its own quirks - so the best course of action is probably to just copy it over (removing the SP_ accordingly.)

Regarding use of intrin.h specifically, I see that the sp version would choose to use it when BOOST_MSVC >= 1600, which would be the case under Clang.

from winapi.

BOOST_MSVC is supposed to be only defined for genuine MSVC. It should not be defined for clang, should it?

from winapi.

You're right, it shouldn't be. Hm.

from winapi.

I confirm that the problem occurs with sp_interlocked.hpp as well. Should be fixed in boostorg/smart_ptr@4eb53db. I still couldn't figure out how to get clang-cl to work under b2 though.

from winapi.

Thanks, Peter. I've incorporated some of your changes in bf9dfd9. Hopefully, it fixes the problem.

from winapi.