[PATCH] D137024: [compiler-rt] Switch from llvm-config to find_package(LLVM)
Martin Storsjö via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 16 14:27:32 PST 2022
mstorsjo added a comment.
In D137024#3931488 <https://reviews.llvm.org/D137024#3931488>, @mgorny wrote:
> Well, I'm certainly not opposed to making all the paths configurable. However, I'm not sure if having CMake file accessible one way or another wouldn't eventually be a necessity. For one thing, I would like to move more common code from standalone build codepaths of individual projects into a dedicated CMake file in LLVM (and while I don't want to speak of others, it seems that there are at least few other people who would like to see something similar done). I suppose this wouldn't be an outright blocker if `llvm/cmake/Modules` directory were present but I'm not 100% sure.
Isn't this two kinda separate concerns? If I build llvm but only install the executables (no libLLVMSupport.a, no cmake files etc), there's no LLVMConfig.cmake. But if I have the full llvm-project monorepo around, all the cmake helper scripts are available to be found in the monorepo - there's just no LLVMConfig.cmake (which is specific for one build of the llvm tools).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137024/new/
https://reviews.llvm.org/D137024
More information about the cfe-commits
mailing list