This is an archive of the discontinued LLVM Phabricator instance.

[openmp] Use llvm GridValues from devicertl
ClosedPublic

Authored by JonChesterfield on Aug 19 2021, 10:28 AM.

Details

Summary

Add include path to the cmakefiles and set the target_impl enums
from the llvm constants instead of copying the values.

Diff Detail

Event Timeline

JonChesterfield requested review of this revision.Aug 19 2021, 10:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 19 2021, 10:28 AM

This diff is relative to D108380 and will not compile without that applied first.

No behaviour change here (the same numbers are set as before, and written to enums to ensure they're compile time constants), but not tagging it NFC as it introduces a dependency from deviceRTL to llvm which is not presently there.

This revision is now accepted and ready to land.Aug 23 2021, 8:58 AM
This revision was landed with ongoing or failed builds.Aug 23 2021, 12:25 PM
This revision was automatically updated to reflect the committed changes.