beanz added a comment.

I think you and I disagree here. General developer workflows don't need to 
include building `all` for every minor change. In my normal workflow I just 
re-run `check-llvm` or `check-clang` over and over again, only building the 
`all` target before I post a patch. With that workflow I only build the library 
once per-patch to ensure that it builds. Which is exactly the goal of not 
having it be included.

If you *really* *really* can't be bothered to ensure that the things we ship 
actually build with your change you can always use the (undocumented, and 
hidden) `CLANG_TOOL_CLANG_SHLIB_BUILD=Off` option. I really don't see a reason 
to add a user-facing option that we don't want people using.


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

Reply via email to