CMake older than 3.20.0 is no longer supported.
This removes work-arounds for no longer supported versions.
Details
- Reviewers
mgorny kwk - Commits
- rGa85f585b1ea9: [LLVM] Removes CMake work-arounds.
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Is this aiming to be exhaustive? For example, there's some still left in libunwind/src/CMakeLists.txt, as came up in https://reviews.llvm.org/D151344
Old CMake mentioned in llvm-owned files:
- https://github.com/llvm/llvm-project/blob/main/llvm/cmake/modules/LLVMCheckLinkerFlag.cmake#L8
- https://github.com/llvm/llvm-project/blob/main/llvm/cmake/modules/AddLLVM.cmake#L2199
There's also a bunch in the docs (primarily https://github.com/llvm/llvm-project/blob/main/llvm/docs/CMakePrimer.rst, but also here & here) though that's probably more work.
llvm/CMakeLists.txt | ||
---|---|---|
101–113 | Marked as up for simplification if CMake >=3.17 |
I feel changing this is out of scope. I agree it would be nice to fix this, but I will not pursue it. Feel free to provide patches for these improvements.
There's also a bunch in the docs (primarily https://github.com/llvm/llvm-project/blob/main/llvm/docs/CMakePrimer.rst, but also here & here) though that's probably more work.
Thanks I'll fix this.
Marked as up for simplification if CMake >=3.17