cmake_minimum_required(VERSION) calls cmake_policy(VERSION),
which sets all policies up to VERSION to NEW.
LLVM started requiring CMake 3.13 last year, so we can remove
a bunch of code setting policies prior to 3.13 to NEW as it
no longer has any effect.
Details
Details
- Reviewers
phosek lebedev.ri ldionne - Group Reviewers
Restricted Project Restricted Project Restricted Project - Commits
- rG480643a95cd1: [CMake] Remove dead code setting policies to NEW
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Event Timeline
Comment Actions
Except for the google-benchmark nit, LGTM. Thanks a lot for the cleanup!
libcxx/utils/google-benchmark/CMakeLists.txt | ||
---|---|---|
3 | I don't think we want to change this. It's a third-party project (which is inconveniently checked-in as-is into our tree..). |
libcxx/utils/google-benchmark/CMakeLists.txt | ||
---|---|---|
3 | Good catch. Did the same for llvm/utils/benchmark/CMakeLists.txt, though that one seems to have the cmake_minimum_required(VERSION) updated to 3.13.4 unlike this one. |
I don't think we want to change this. It's a third-party project (which is inconveniently checked-in as-is into our tree..).