[PATCH] D55791: [libcxx] Simplify conditional definition of _LIBCPP_HAS_NO_ALIGNED_ALLOCATION
Louis Dionne via Phabricator
reviews at reviews.llvm.org
Mon Dec 17 14:36:12 PST 2018
ldionne added a comment.
I'm putting up a review for this simple change in case the intent was to allow users to define `_LIBCPP_HAS_NO_ALIGNED_ALLOCATION` before including a libc++ header. I don't expect that was the intent, but just in case.
If that happens to be true, I think that intent should be documented and a CMake `#define` should be added. I'd also like to understand why that use case is something we need to support.
Repository:
rCXX libc++
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55791/new/
https://reviews.llvm.org/D55791
More information about the libcxx-commits
mailing list