[PATCH] D64580: cmake: Add INSTALL_WITH_TOOLCHAIN option to add_*_library macros

2019-07-12 Thread Tom Stellard via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL365902: cmake: Add INSTALL_WITH_TOOLCHAIN option to add_*_library macros (authored by tstellar, committed by ). Changed prior to commit: https://reviews.llvm.org/D64580?vs=209285=209482#toc

[PATCH] D64580: cmake: Add INSTALL_WITH_TOOLCHAIN option to add_*_library macros

2019-07-11 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai added a comment. Nice. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64580/new/ https://reviews.llvm.org/D64580 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D64580: cmake: Add INSTALL_WITH_TOOLCHAIN option to add_*_library macros

2019-07-11 Thread Chris Bieneman via Phabricator via cfe-commits
beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64580/new/ https://reviews.llvm.org/D64580 ___

[PATCH] D64580: cmake: Add INSTALL_WITH_TOOLCHAIN option to add_*_library macros

2019-07-11 Thread Tom Stellard via Phabricator via cfe-commits
tstellar created this revision. tstellar added reviewers: beanz, smeenai. Herald added subscribers: dexonsmith, steven_wu, mgorny, mehdi_amini. Herald added projects: clang, LLVM. This will simplify the macros by allowing us to remove the hard-coded list of libraries that should be installed when