EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land.
One concern I have is being able to specify libraries in the correct order when we have two different lists. However I'm not concerned enough not to try it. LGTM. ================ Comment at: lib/CMakeLists.txt:43 + # Apple re-exports libc++abi in libc++, so don't make it public + add_library_flags("${LIBCXX_CXX_ABI_LIBRARY}") +else() ---------------- Urg. I think this whole setup is borked on Apple. This was an existing issue though, so you don't have to fix it. this LGTM for now. https://reviews.llvm.org/D25008 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits