This revision was automatically updated to reflect the committed changes.
Closed by commit rGac2c5af67f03: [OPENMP] Fix mixture of omp and clang pragmas
(authored by hishiguro, committed by kawashima-fj).
Changed prior to commit:
https://reviews.llvm.org/D79921?vs=265450&id=265667#toc
Reposito
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
LG
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79921/new/
https://reviews.llvm.org/D79921
___
cfe-commits mailing list
cfe-commits@l
hishiguro updated this revision to Diff 265450.
hishiguro added a comment.
Thank you for your comment. I updated source.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79921/new/
https://reviews.llvm.org/D79921
Files:
clang/lib/CodeGen/CGStmtOpenMP.cpp
clang/test/OpenMP/omp_with_loo
ABataev added inline comments.
Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:1735
+ // If attributes are attached, push to the basic block with them.
+ const OMPExecutableDirective *OMPED = dyn_cast(&S);
+ const CapturedStmt *ICS = OMPED->getInnermostCapturedStmt();
-
hishiguro updated this revision to Diff 264836.
hishiguro retitled this revision from "[OpenMP] Fix omp and clang pragmas" to
"[OPENMP] Fix mixture of omp and clang pragmas".
hishiguro added a comment.
Herald added a reviewer: a.sidorin.
I checked that your comments are correct. I modified the so
