Rename LLVM_CONFIG_EXE to LLVM_CONFIG_PATH, and avoid building it if
passed in by user. This is the same way CLANG_TABLEGEN and
LLVM_TABLEGEN are handled, e.g., when -DLLVM_OPTIMIZED_TABLEGEN=ON is
passed.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
I think LLVM_CONFIG_PATH is a more appropriate as that name is already used by other LLVM projects exactly for this purpose.
Comment Actions
This seems to work better then what I have in D41580 for llvm_config.
LGTM.
We should try to get this back ported to the 6.0 branch before release.
Comment Actions
Add LLVM_CONFIG_PATH to the cache instead of using PARENT_SCOPE, since
PARENT_SCOPE only goes up one level -- tools directory in this case --
and won't be visible to llvm_ExternalProject_Add otherwise.