This is an archive of the discontinued LLVM Phabricator instance.

Remove obsolete policy settings
ClosedPublic

Authored by steveire on Aug 7 2018, 2:38 PM.

Details

Summary

The line

cmake_minimum_required(VERSION 3.4.3)

already has the effect of setting to NEW all policies present in that
release:

https://cmake.org/cmake/help/v3.4/manual/cmake-policies.7.html

Diff Detail

Repository
rL LLVM

Event Timeline

steveire created this revision.Aug 7 2018, 2:38 PM
chapuni added a subscriber: chapuni.Aug 7 2018, 3:46 PM

ping? This should be a striaghtforward lgtm from anyone.

dblaikie accepted this revision.Aug 9 2018, 12:45 PM

looks good, thanks! (do you have commit rights - or would you like me to commit this for you?)

This revision is now accepted and ready to land.Aug 9 2018, 12:45 PM

I have commit rights, thanks!

This revision was automatically updated to reflect the committed changes.
smeenai added inline comments.
llvm/trunk/CMakeLists.txt
16

This removal was incorrect. https://cmake.org/cmake/help/v3.4/policy/CMP0056.html says "Unlike most policies, CMake version 3.4.3 does not warn by default when this policy is not set and simply uses OLD behavior (and even the latest version's documentaion says the same).

In general, @beanz, @phosek, @mgorny and I make good reviewers for CMake changes.

Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2019, 6:34 PM
smeenai added inline comments.Apr 17 2019, 6:36 PM
llvm/trunk/CMakeLists.txt
16

Ah, nevermind, the document you linked seems more authoritative than the individual policy page's one. That's confusing documentation though.