[libcxx-commits] [PATCH] D56913: decoupling Freestanding atomic<T> from libatomic.a

Olivier Giroux via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Feb 6 21:15:12 PST 2019


__simt__ updated this revision to Diff 185704.
__simt__ marked an inline comment as done.
__simt__ added a comment.

In this version:

1. Macros moved to `__config`.
2. Redundant static/`_LIBCPP_INLINE_VISIBILITY` simplified.
3. Redundant inline/template simplified.
4. Applied `_LIBCPP_DEFAULT` where it is sufficient for c++03.
5. Scopes removed.
6. A few cut-n-paste mistakes fixed.

Potentially the last major question I see is whether the new code needs to support c++03 or not.


Repository:
  rCXX libc++

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56913/new/

https://reviews.llvm.org/D56913

Files:
  libcxx/include/__config
  libcxx/include/atomic

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56913.185704.patch
Type: text/x-patch
Size: 52629 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20190207/dcbdbb2f/attachment-0001.bin>


More information about the libcxx-commits mailing list