[PATCH] D42436: libcxx: Allow auto-linking to be disabled with a macro.

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 23 15:16:08 PST 2018


compnerd added a comment.

I'm fine with the idea of this knob, I just would prefer a slightly different implementation.  Could you wrap the entire block in the `defined(_LIBCPP_NO_AUTO_LINK)` rather than adding the additional conditional?  I feel like it makes it easier to read.


https://reviews.llvm.org/D42436





More information about the llvm-commits mailing list