[PATCH] D81483: [Clang][OpenMP][OMPBuilder] (3/4) Privatize `parallel` for `OMPBuilder`

[Fady Ghanim via Phabricator via cfe-commits]

fghanim marked 3 inline comments as done.
fghanim added inline comments.


================
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:1554
                                      llvm::SmallVectorImpl<llvm::Value *> &) {}
 
+bool CodeGenFunction::OMPBuilderCBHelpers::EmitOMPCopyinClause(
----------------
Note to reviewers:
This keeps most of old implementation of `copyin` clause, except it uses part of the `OMPBuilder` now. Changes are noted below.


================
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:1582
+          assert(CGF.CapturedStmtInfo->lookup(VD) &&
+                 "Copyin threadprivates should have been captured!");
+          const auto *VDCanon = VD->getCanonicalDecl();
----------------
added following few lines


================
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:1600
+        }
+
+        // Get the address of the threadprivate variable.
----------------
Wherever needed - changed the following section to use `OMPBuilder`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81483/new/

https://reviews.llvm.org/D81483