This is an archive of the discontinued LLVM Phabricator instance.

[MLIR] Fix MLIR_MAIN_[SRC|INCLUDE]_DIR variables
ClosedPublic

Authored by ekatz on Apr 11 2020, 7:58 AM.

Details

Summary

Define MLIR_MAIN_INCLUDE_DIR, as it was not set anywhere.
Set MLIR_MAIN_SRC_DIR to the actual "source directory", and not the "include directory" (as currently set).

Diff Detail

Event Timeline

ekatz created this revision.Apr 11 2020, 7:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 11 2020, 7:58 AM
ekatz added a comment.Apr 11 2020, 8:00 AM

Note that MLIR_MAIN_INCLUDE_DIR is used in almost all (if not all) the CMakeLists.txt in the project.

mehdi_amini accepted this revision.Apr 11 2020, 10:46 AM
This revision is now accepted and ready to land.Apr 11 2020, 10:46 AM
This revision was automatically updated to reflect the committed changes.