This is an archive of the discontinued LLVM Phabricator instance.

[OPENMP52] Support Support omp_cur_iteration modifier for doacross clause.
ClosedPublic

Authored by jyu2 on Jul 5 2023, 5:29 PM.

Details

Summary

This is just syntax to make it easier for the user. It doesn't add any
new functionality.

for
doacross(sink: omp_cur_iteration - 1)
Equivalent to
doacross(sink: ConterVar - 1, ...)

doacross(source: omp_cur_iteration)
Equivalent to
doacross(source)

And restriction is:
OMP5.2 p.327

If vector is specified with the omp_cur_iteration keyword and with
sink as the dependence-type then it must be omp_cur_iteration - 1.

If vector is specified with source as the dependence-type then it must be
omp_cur_iteration.

Diff Detail

Event Timeline

jyu2 created this revision.Jul 5 2023, 5:29 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 5 2023, 5:29 PM
jyu2 requested review of this revision.Jul 5 2023, 5:29 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 5 2023, 5:29 PM
jyu2 updated this revision to Diff 537553.Jul 5 2023, 5:32 PM

Fix typo.

ABataev added inline comments.Jul 6 2023, 5:57 AM
clang/lib/AST/OpenMPClause.cpp
2516–2523

switch?

jyu2 updated this revision to Diff 537749.Jul 6 2023, 8:47 AM

Thanks Alexey for the review. This is address his comment.

clang/lib/AST/OpenMPClause.cpp
2516–2523

Thanks. Changed!.

This revision is now accepted and ready to land.Jul 6 2023, 9:27 AM
This revision was landed with ongoing or failed builds.Jul 6 2023, 11:53 AM
This revision was automatically updated to reflect the committed changes.