EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land.
LGTM modulo inline comments. ================ Comment at: CMakeLists.txt:310 @@ -309,4 +309,3 @@ -# Remove flags that may have snuck in. -remove_flags(-DNDEBUG -UNDEBUG -D_DEBUG - -stdlib=libc++ -stdlib=libstdc++ -lc++abi -m32) +if (LIBCXX_STANDALONE_BUILD) + # Remove flags that may have snuck in. ---------------- We always want to strip `-stdlib=<foo>` because it's just nonsense to enable the STL while building the STL. https://reviews.llvm.org/D24809 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits