[PATCH] D57535: [CMake] Use LLVM_ENABLE_PROJECTS as the "single source" of truth when used.

Shoaib Meenai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 8 18:22:07 PST 2019


smeenai added a comment.

In D57535#1388620 <https://reviews.llvm.org/D57535#1388620>, @delcypher wrote:

> In D57535#1387792 <https://reviews.llvm.org/D57535#1387792>, @mehdi_amini wrote:
>
> > Could we sanitize the list of LLVM_ENABLE_PROJECTS and error for unknown entries?
>
>
> I'm fine with this. @smeenai are you doing this or would you like me to put up a patch for this?


Sorry for the late reply. Feel free to put up a patch for that. Just watch out for clang-tools-extra; we can't include it in `LLVM_ALL_PROJECTS` (see rL353354 <https://reviews.llvm.org/rL353354>), but we also shouldn't error for it.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57535/new/

https://reviews.llvm.org/D57535





More information about the llvm-commits mailing list