[llvm] [cmake] Move LLVM_ENABLE_PIC cmake option to HandleLLVMOptions.cmake (PR #84750)

Kristina Bessonova via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 12 13:05:24 PDT 2024


chbessonova wrote:

@compnerd thank you for looking at this!
Unfortunately, I didn't noticed that this change creates a loop dependency between `HandleLLVMOptions.cmake` and `llvm/cmake/config-ix.cmake` which included earlier and requires `LLVM_ENABLE_PIC` to be defined.
We might need a specific cmake module to define options common among LLVM projects (I'm pretty sure that `LLVM_ENABLE_PIC` is not an only option that should be exposed for standalone builds) to make this clear and transparent. I'll think about this a day or two and return with a better solution.


https://github.com/llvm/llvm-project/pull/84750


More information about the llvm-commits mailing list