[PATCH] D141528: [Clang][OpenMP] Fix loop directive nested inside a parallel

doru1004 closed this revision. doru1004 added a comment. Commit: 1407dbeabcfed114f0918b022365d03713dac028 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141528/new/ https://reviews.llvm.org/D141528 __

[PATCH] D141528: [Clang][OpenMP] Fix loop directive nested inside a parallel

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/D141528/new/ https://reviews.llvm.org/D141528 ___ cfe-commits mailing list cfe-commits@l

[PATCH] D141528: [Clang][OpenMP] Fix loop directive nested inside a parallel

doru1004 added a comment. ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141528/new/ https://reviews.llvm.org/D141528 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D141528: [Clang][OpenMP] Fix loop directive nested inside a parallel

doru1004 updated this revision to Diff 489105. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141528/new/ https://reviews.llvm.org/D141528 Files: clang/lib/CodeGen/CGStmtOpenMP.cpp clang/test/OpenMP/nested_loop_codegen.cpp Index: clang/test/OpenMP/nested_loop_codegen.cpp

[PATCH] D141528: [Clang][OpenMP] Fix loop directive nested inside a parallel

doru1004 updated this revision to Diff 489035. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141528/new/ https://reviews.llvm.org/D141528 Files: clang/lib/CodeGen/CGStmtOpenMP.cpp clang/test/OpenMP/nested_loop_codegen.cpp Index: clang/test/OpenMP/nested_loop_codegen.cpp

[PATCH] D141528: [Clang][OpenMP] Fix loop directive nested inside a parallel

jdoerfert added a comment. Drive by. Comment at: clang/test/OpenMP/nested_loop_codegen.cpp:32 + +int internal_decl() { + int i, res; Internal? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141528/new/ https://r

[PATCH] D141528: [Clang][OpenMP] Fix loop directive nested inside a parallel

ABataev added inline comments. Comment at: clang/lib/CodeGen/CGStmtOpenMP.cpp:7865 +const Stmt *CS = cast(S.getAssociatedStmt())->getCapturedStmt(); +if (CS->getStmtClass() == Stmt::ForStmtClass) { + const ForStmt &ForS = cast(*CS); ``` if (const au