smeenai accepted this revision. smeenai added a comment. This revision is now accepted and ready to land.
With the newer minimum CMake version, we can eliminate a bunch of the policy settings as well, but that can be a follow-up (especially given this isn't gonna ship for a few months). (Another followup would be to add a `max` version to our `cmake_minimum_required`, to streamline policy settings even further. See https://cmake.org/cmake/help/latest/command/cmake_policy.html for details.) ================ Comment at: llvm/docs/CMakePrimer.rst:57 - cmake_minimum_required(VERSION 3.2) + cmake_minimum_required(VERSION 3.15) project(HelloWorld) ---------------- Is the higher minimum here for future-proofing? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78648/new/ https://reviews.llvm.org/D78648 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
