This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Fix building using LLVM_ENABLE_RUNTIMES
ClosedPublic

Authored by ggeorgakoudis on Jan 25 2021, 2:55 PM.

Details

Summary

Fix when time profiling is enabled.

Related to: D94855

Diff Detail

Event Timeline

ggeorgakoudis created this revision.Jan 25 2021, 2:55 PM
ggeorgakoudis requested review of this revision.Jan 25 2021, 2:55 PM
Herald added a project: Restricted Project. · View Herald Transcript
JonChesterfield accepted this revision.Jan 25 2021, 3:29 PM

LGTM, thanks. Building via LLVM_ENABLE_RUNTIMES is what we're encouraging for GPU offloading.

This revision is now accepted and ready to land.Jan 25 2021, 3:29 PM

This should probably be applied to the 12.0 release branch as well

This revision was landed with ongoing or failed builds.Jan 27 2021, 6:44 AM
This revision was automatically updated to reflect the committed changes.

This should probably be applied to the 12.0 release branch as well

@JonChesterfield I landed, is there anything else I should do?

This should probably be applied to the 12.0 release branch as well

@JonChesterfield I landed, is there anything else I should do?

https://reviews.llvm.org/D95545#2525853