This is an archive of the discontinued LLVM Phabricator instance.

[LLVM] Removes CMake work-arounds.
ClosedPublic

Authored by Mordante on Jun 4 2023, 8:46 AM.

Details

Summary

CMake older than 3.20.0 is no longer supported.
This removes work-arounds for no longer supported versions.

Diff Detail

Event Timeline

Mordante created this revision.Jun 4 2023, 8:46 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 4 2023, 8:46 AM
Mordante requested review of this revision.Jun 4 2023, 8:46 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 4 2023, 8:46 AM

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

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

This should be everything in the LLVM directory, the range [D152099, D152103) contains the other directories.

h-vetinari added a comment.EditedJun 5 2023, 5:25 PM
llvm/CMakeLists.txt
101–113

Marked as up for simplification if CMake >=3.17

Mordante planned changes to this revision.Jun 5 2023, 11:09 PM

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.

kwk accepted this revision.Jun 6 2023, 2:29 AM

LGTM

This revision was not accepted when it landed; it landed in state Changes Planned.Jun 7 2023, 9:04 AM
This revision was automatically updated to reflect the committed changes.