This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Fixed an issue that wrong LLVM headers might be included when building libomptarget
ClosedPublic

Authored by tianshilei1992 on Dec 22 2020, 5:19 PM.

Details

Summary

Wrong LLVM headers might be included if we don't set include_directories
to a right place. This will cause a compilation error if LLVM is installed in
system directories.

Diff Detail

Event Timeline

tianshilei1992 created this revision.Dec 22 2020, 5:19 PM
tianshilei1992 requested review of this revision.Dec 22 2020, 5:19 PM
Herald added a project: Restricted Project. · View Herald Transcript
jdoerfert accepted this revision.Jan 6 2021, 1:39 PM

LGTM.

This revision is now accepted and ready to land.Jan 6 2021, 1:39 PM