ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
https://reviews.llvm.org/D22691
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
sfantao added a comment.
Hi Alexey,
Thanks for the review!
Comment at: lib/CodeGen/CGStmtOpenMP.cpp:3410-3411
@@ +3409,4 @@
+ llvm::DenseSet EmittedAsFirstprivate;
+ CGCapturedStmtInfo CapturesInfo(cast(*D.getAssociatedStmt()));
+ for (const auto *C : D.getClausesOfKind()) {
sfantao updated this revision to Diff 65810.
sfantao marked 8 inline comments as done.
sfantao added a comment.
- Remove unnecessary brief directives and refactor target data privatization
code genenration.
https://reviews.llvm.org/D22691
Files:
include/clang/AST/OpenMPClause.h
lib/AST/Ope
ABataev requested changes to this revision.
This revision now requires changes to proceed.
Comment at: include/clang/AST/OpenMPClause.h:4252
@@ -4237,1 +4251,3 @@
+
+ /// \brief Build clause with number of variables \a NumVars.
///
No \brief
sfantao updated this revision to Diff 65123.
sfantao added a comment.
- Use the correct enumerator for return pointer, as specified in the design
document.
https://reviews.llvm.org/D22691
Files:
include/clang/AST/OpenMPClause.h
lib/AST/OpenMPClause.cpp
lib/CodeGen/CGOpenMPRuntime.cpp
l
