[OpenMP] Use assumptions during ICV tracking
ClosedPublic
Actions

Authored by jdoerfert on Nov 24 2020, 11:42 AM.

Details

Reviewers

sstefan1
JonChesterfield
jhuber6
ggeorgakoudis

Commits

rGdcaec81211be: [OpenMP] Use assumptions during ICV tracking

Summary

The OpenMP 5.1 assumptions no_openmp and no_openmp_routines allow us
to ignore calls that would otherwise prevent ICV tracking.

Once we track more ICVs we might need to distinguish the ones that could
be impacted even with no_openmp_routines.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

jdoerfert created this revision.Nov 24 2020, 11:42 AM

Herald added a project: Restricted Project. · View Herald TranscriptNov 24 2020, 11:42 AM

Herald added subscribers: guansong, bollu, hiraditya, yaxunl. · View Herald Transcript

jdoerfert requested review of this revision.Nov 24 2020, 11:42 AM

jdoerfert added a reviewer: ggeorgakoudis.Nov 24 2020, 11:47 AM

Harbormaster completed remote builds in B79995: Diff 307419.Nov 24 2020, 12:19 PM

Cool :)

LGTM

This revision is now accepted and ready to land.Nov 24 2020, 12:36 PM

Closed by commit rGdcaec81211be: [OpenMP] Use assumptions during ICV tracking (authored by jdoerfert). · Explain WhyDec 15 2020, 2:52 PM

This revision was automatically updated to reflect the committed changes.

jdoerfert added a commit: rGdcaec81211be: [OpenMP] Use assumptions during ICV tracking.

Revision Contents
Changeset List

Path

Size

llvm/

lib/

Transforms/

IPO/

OpenMPOpt.cpp

3 lines

test/

Transforms/

OpenMP/

icv_tracking.ll

44 lines

Diff 312043

View Options

llvm/lib/Transforms/IPO/OpenMPOpt.cpp