diff options
| author | Alexey Bataev <a.bataev@hotmail.com> | 2019-07-17 18:03:39 +0000 |
|---|---|---|
| committer | Alexey Bataev <a.bataev@hotmail.com> | 2019-07-17 18:03:39 +0000 |
| commit | 48f5a43bcc3b128aca54b99b565595a5fa6ad166 (patch) | |
| tree | cf3b9a21568353265a5e339b659bc21bbba29c30 /clang | |
| parent | f90d3dff6edc906766cfe3196d6ac1b6d76e9c5a (diff) | |
| download | bcm5719-llvm-48f5a43bcc3b128aca54b99b565595a5fa6ad166.tar.gz bcm5719-llvm-48f5a43bcc3b128aca54b99b565595a5fa6ad166.zip | |
[OPENMP]Fix PR42632: crash on the analysis of the OpenMP constructs.
Fixed processing of the CapturedStmt children to fix the crash of the
OpenMP constructs during analysis.
llvm-svn: 366357
Diffstat (limited to 'clang')
| -rw-r--r-- | clang/lib/AST/ParentMap.cpp | 12 | ||||
| -rw-r--r-- | clang/test/Analysis/openmp-unsupported.c | 4 |
2 files changed, 16 insertions, 0 deletions
diff --git a/clang/lib/AST/ParentMap.cpp b/clang/lib/AST/ParentMap.cpp index e09b5bbe75f..2ff5c9d8aeb 100644 --- a/clang/lib/AST/ParentMap.cpp +++ b/clang/lib/AST/ParentMap.cpp @@ -83,6 +83,18 @@ static void BuildParentMap(MapTy& M, Stmt* S, } break; } + case Stmt::CapturedStmtClass: + for (Stmt *SubStmt : S->children()) { + if (SubStmt) { + M[SubStmt] = S; + BuildParentMap(M, SubStmt, OVMode); + } + } + if (Stmt *SubStmt = cast<CapturedStmt>(S)->getCapturedStmt()) { + M[SubStmt] = S; + BuildParentMap(M, SubStmt, OVMode); + } + break; default: for (Stmt *SubStmt : S->children()) { if (SubStmt) { diff --git a/clang/test/Analysis/openmp-unsupported.c b/clang/test/Analysis/openmp-unsupported.c index 7e363eecbaa..b2e1a1b0217 100644 --- a/clang/test/Analysis/openmp-unsupported.c +++ b/clang/test/Analysis/openmp-unsupported.c @@ -4,4 +4,8 @@ void openmp_parallel_crash_test() { #pragma omp parallel ; +#pragma omp parallel for + for (int i = 0; i < 8; ++i) + for (int j = 0, k = 0; j < 8; ++j) + ; } |

