This is an archive of the discontinued LLVM Phabricator instance.

[flang][OpenMP] Support for privatization in common block
ClosedPublic

Authored by NimishMishra on Jul 24 2023, 6:20 AM.

Details

Summary

This patch provides support for usage of common block in private/firstprivate and lastprivate clauses/

Depends on: D127215

Diff Detail

Event Timeline

NimishMishra created this revision.Jul 24 2023, 6:20 AM
NimishMishra requested review of this revision.Jul 24 2023, 6:20 AM
flang/lib/Lower/OpenMP.cpp
388–390

Did you miss including the copyFirstPrivateSymbol in the else block?

NimishMishra added inline comments.
flang/lib/Lower/OpenMP.cpp
388–390

Yes. Thank you for the spot.

LG.

flang/lib/Lower/OpenMP.cpp
404–405

Nit: Add braces here.

This revision is now accepted and ready to land.Jul 27 2023, 4:54 AM
This revision was landed with ongoing or failed builds.Jul 31 2023, 4:17 AM
This revision was automatically updated to reflect the committed changes.
flang/lib/Lower/OpenMP.cpp