smeenai added a comment. A couple of things I noticed while writing this patch:
- clang guards its install target creation with `CMAKE_CONFIGURATION_TYPES`, whereas LLVM uses `LLVM_ENABLE_IDE`. Should clang be switched over to be consistent with LLVM? - I realize we create the `install-clang-libraries-stripped` target (courtesy of `add_llvm_install_targets`), but don't actually add any dependencies to it, so it's a no-op. `llvm-libraries` has this problem too. I was thinking of just making `install-clang-libraries-stripped` synonymous with `install-clang-libraries`, because a lot of these libraries will be static, and stripping them makes them unusable since it removes their symbol tables. The target name will be a lie, but the behavior seems more useful. What do you think? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58269/new/ https://reviews.llvm.org/D58269 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
