jvesely added a comment. In D66068#1625478 <https://reviews.llvm.org/D66068#1625478>, @beanz wrote:
> I generally am not a fan of adding more and more options. As long as you're > not linking the library it won't be generated by any of the check targets. > With the llvm dylib we've had many issues over the years where changes to > LLVM break building the dylib and many developers don't build it, so we have > to wait for a bot to catch it. Making the clang dylib build as part of `all` > in every possible build configuration is a recognition that this is something > we ship and we should ensure works. I've no problem with it staying ON by default. I just don't see a reason to make every minor modification rebuild last 5-6mins instead of 30s, for something I don't want, need or use. Not everyone is a billion-dollar corporation with a dedicated build farm. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66068/new/ https://reviews.llvm.org/D66068 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
