[PATCH] D10406: [libcxx] Add atomic_support.h header to src that handles needed atomic operations.

Eric Fiselier eric at efcs.ca
Mon Jul 6 14:33:41 PDT 2015



1. Fix increment/decrement consistency to fix PR22803.
2. Put functions in `atomic_support.h` inside  an inline namespace.
3. Generate a build warning on MSVC when atomics aren't supported.
4. Fix bad assertion inside shared_ptr race condition test case.

I don't think we should make any attempt to support GCC and Clang versions that do not support the `__atomic*` builtins. This would limit *building* (not using) libc++ to GCC 4.7 and beyond and clang 3.3 (possible earlier). I don't have an issue with this.


http://reviews.llvm.org/D10406

Files:
  include/memory
  include/mutex
  include/thread
  src/memory.cpp
  src/mutex.cpp
  src/support/atomic_support.h
  test/libcxx/utilities/memory/util.smartptr/race_condition.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10406.29126.patch
Type: text/x-patch
Size: 12617 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150706/f0b4a38b/attachment.bin>


More information about the cfe-commits mailing list