This is an archive of the discontinued LLVM Phabricator instance.

[Clang][OpenMP][OMPBuilder] (3/4) Privatize `parallel` for `OMPBuilder`
Needs ReviewPublic

Authored by fghanim on Jun 9 2020, 10:06 AM.

Details

Reviewers
jdoerfert
Summary

Adding support / implementation for copyin in OMPBuilder's
implementation of omp parallel

Diff Detail

Event Timeline

fghanim created this revision.Jun 9 2020, 10:06 AM
fghanim marked 3 inline comments as done.Jun 9 2020, 10:23 AM
fghanim added inline comments.
clang/lib/CodeGen/CGStmtOpenMP.cpp
1554

Note to reviewers:
This keeps most of old implementation of copyin clause, except it uses part of the OMPBuilder now. Changes are noted below.

1582

added following few lines

1600

Wherever needed - changed the following section to use OMPBuilder