[PATCH] D85140: [RFC] Factor out repetitive cmake patterns for llvm-style projects

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

Something you don't make very explicit in the description of this patch is that beyond a "refactoring" this is also changing the behavior by *duplicating* the options for each subproject. 
For example building with `-DLLVM_INCLUDE_EXAMPLES` won't be enough to include the examples in MLIR, the user must also provide `-DMLIR_INCLUDE_EXAMPLES`. Same for every other options.

I can see how this provide more flexibility to the user configuring the build, but this flexibility comes also with a more complex surface. For example when building LLVM+MLIR, passing `-DLLVM_INSTALL_TOOLCHAIN_ONLY=ON` isn't enough and you need also `-DMLIR_INSTALL_TOOLCHAIN_ONLY=ON`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85140