This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Change OpenMPOpt to check openmp metadata
ClosedPublic

Authored by jhuber6 on May 13 2021, 10:42 AM.

Details

Summary

The metadata added in D102361 introduces a module flag that we can check
to determine if the module was compiled with -fopenmp enables. We can
now check for the precense of this instead of scanning the call graph
for OpenMP runtime functions.

Depends on D102361

Diff Detail

Event Timeline

jhuber6 created this revision.May 13 2021, 10:42 AM
jhuber6 requested review of this revision.May 13 2021, 10:42 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 13 2021, 10:42 AM
jhuber6 updated this revision to Diff 354397.Jun 24 2021, 4:45 PM

Updating.

jdoerfert accepted this revision.Jun 25 2021, 12:31 PM

LG, one nit.

llvm/lib/Transforms/IPO/OpenMPOpt.cpp
2748–2749

We don't need this anymore. Remove the entire function.

This revision is now accepted and ready to land.Jun 25 2021, 12:31 PM
jhuber6 updated this revision to Diff 354583.Jun 25 2021, 12:40 PM

Removing unused call.

This revision was landed with ongoing or failed builds.Jun 25 2021, 1:34 PM
This revision was automatically updated to reflect the committed changes.