This is an archive of the discontinued LLVM Phabricator instance.

[MLIR] Removes CMake work-arounds.
ClosedPublic

Authored by Mordante on Jun 4 2023, 8:43 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:43 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 4 2023, 8:43 AM
Mordante requested review of this revision.Jun 4 2023, 8:43 AM
mehdi_amini accepted this revision.Jun 5 2023, 9:55 AM
This revision is now accepted and ready to land.Jun 5 2023, 9:55 AM
h-vetinari added inline comments.
mlir/tools/mlir-vulkan-runner/CMakeLists.txt
16–17
h-vetinari added inline comments.Jun 5 2023, 5:16 PM
mlir/cmake/modules/MLIRDetectPythonEnv.cmake
14–19

Not sure if removing mentions of old CMake is in scope as well. There may occasionally be a good reason to keep such comments, but it does pollute the search results when looking for old CMake

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.

Mordante marked 2 inline comments as done.Jun 7 2023, 8:46 AM
Mordante added inline comments.
mlir/cmake/modules/MLIRDetectPythonEnv.cmake
14–19

I prefer to leave that cleanup to the MLIR maintainers. They might have intended to to more than just removing the comment.

This revision was not accepted when it landed; it landed in state Changes Planned.Jun 7 2023, 8:47 AM
This revision was automatically updated to reflect the committed changes.
Mordante marked an inline comment as done.