diff options
author | Alexey Bataev <a.bataev@hotmail.com> | 2014-07-24 06:46:57 +0000 |
---|---|---|
committer | Alexey Bataev <a.bataev@hotmail.com> | 2014-07-24 06:46:57 +0000 |
commit | 459dec0ca2a765fc7fa6bc24b662a5e10df18e97 (patch) | |
tree | eacd841808eb3ec53909056748ac2af85fcbd208 /clang/lib/Sema/SemaOpenMP.cpp | |
parent | 65f8732c146f92a6960f352e12959b613d32e848 (diff) | |
download | bcm5719-llvm-459dec0ca2a765fc7fa6bc24b662a5e10df18e97.tar.gz bcm5719-llvm-459dec0ca2a765fc7fa6bc24b662a5e10df18e97.zip |
[OPENMP] Initial parsing and sema analysis for clause 'capture' in 'atomic' directive.
llvm-svn: 213842
Diffstat (limited to 'clang/lib/Sema/SemaOpenMP.cpp')
-rw-r--r-- | clang/lib/Sema/SemaOpenMP.cpp | 38 |
1 files changed, 31 insertions, 7 deletions
diff --git a/clang/lib/Sema/SemaOpenMP.cpp b/clang/lib/Sema/SemaOpenMP.cpp index 9c05e9204e6..5bbed680113 100644 --- a/clang/lib/Sema/SemaOpenMP.cpp +++ b/clang/lib/Sema/SemaOpenMP.cpp @@ -2392,7 +2392,8 @@ StmtResult Sema::ActOnOpenMPAtomicDirective(ArrayRef<OMPClause *> Clauses, SourceLocation AtomicKindLoc; for (auto *C : Clauses) { if (C->getClauseKind() == OMPC_read || C->getClauseKind() == OMPC_write || - C->getClauseKind() == OMPC_update) { + C->getClauseKind() == OMPC_update || + C->getClauseKind() == OMPC_capture) { if (AtomicKind != OMPC_unknown) { Diag(C->getLocStart(), diag::err_omp_atomic_several_clauses) << SourceRange(C->getLocStart(), C->getLocEnd()); @@ -2404,25 +2405,36 @@ StmtResult Sema::ActOnOpenMPAtomicDirective(ArrayRef<OMPClause *> Clauses, } } } + auto Body = CS->getCapturedStmt(); if (AtomicKind == OMPC_read) { - if (!isa<Expr>(CS->getCapturedStmt())) { - Diag(CS->getCapturedStmt()->getLocStart(), + if (!isa<Expr>(Body)) { + Diag(Body->getLocStart(), diag::err_omp_atomic_read_not_expression_statement); return StmtError(); } } else if (AtomicKind == OMPC_write) { - if (!isa<Expr>(CS->getCapturedStmt())) { - Diag(CS->getCapturedStmt()->getLocStart(), + if (!isa<Expr>(Body)) { + Diag(Body->getLocStart(), diag::err_omp_atomic_write_not_expression_statement); return StmtError(); } } else if (AtomicKind == OMPC_update || AtomicKind == OMPC_unknown) { - if (!isa<Expr>(CS->getCapturedStmt())) { - Diag(CS->getCapturedStmt()->getLocStart(), + if (!isa<Expr>(Body)) { + Diag(Body->getLocStart(), diag::err_omp_atomic_update_not_expression_statement) << (AtomicKind == OMPC_update); return StmtError(); } + } else if (AtomicKind == OMPC_capture) { + if (isa<Expr>(Body) && !isa<BinaryOperator>(Body)) { + Diag(Body->getLocStart(), + diag::err_omp_atomic_capture_not_expression_statement); + return StmtError(); + } else if (!isa<Expr>(Body) && !isa<CompoundStmt>(Body)) { + Diag(Body->getLocStart(), + diag::err_omp_atomic_capture_not_compound_statement); + return StmtError(); + } } getCurFunction()->setHasBranchProtectedScope(); @@ -2472,6 +2484,7 @@ OMPClause *Sema::ActOnOpenMPSingleExprClause(OpenMPClauseKind Kind, Expr *Expr, case OMPC_read: case OMPC_write: case OMPC_update: + case OMPC_capture: case OMPC_unknown: llvm_unreachable("Clause is not allowed."); } @@ -2677,6 +2690,7 @@ OMPClause *Sema::ActOnOpenMPSimpleClause( case OMPC_read: case OMPC_write: case OMPC_update: + case OMPC_capture: case OMPC_unknown: llvm_unreachable("Clause is not allowed."); } @@ -2795,6 +2809,7 @@ OMPClause *Sema::ActOnOpenMPSingleExprWithArgClause( case OMPC_read: case OMPC_write: case OMPC_update: + case OMPC_capture: case OMPC_unknown: llvm_unreachable("Clause is not allowed."); } @@ -2883,6 +2898,9 @@ OMPClause *Sema::ActOnOpenMPClause(OpenMPClauseKind Kind, case OMPC_update: Res = ActOnOpenMPUpdateClause(StartLoc, EndLoc); break; + case OMPC_capture: + Res = ActOnOpenMPCaptureClause(StartLoc, EndLoc); + break; case OMPC_if: case OMPC_final: case OMPC_num_threads: @@ -2944,6 +2962,11 @@ OMPClause *Sema::ActOnOpenMPUpdateClause(SourceLocation StartLoc, return new (Context) OMPUpdateClause(StartLoc, EndLoc); } +OMPClause *Sema::ActOnOpenMPCaptureClause(SourceLocation StartLoc, + SourceLocation EndLoc) { + return new (Context) OMPCaptureClause(StartLoc, EndLoc); +} + OMPClause *Sema::ActOnOpenMPVarListClause( OpenMPClauseKind Kind, ArrayRef<Expr *> VarList, Expr *TailExpr, SourceLocation StartLoc, SourceLocation LParenLoc, SourceLocation ColonLoc, @@ -3000,6 +3023,7 @@ OMPClause *Sema::ActOnOpenMPVarListClause( case OMPC_read: case OMPC_write: case OMPC_update: + case OMPC_capture: case OMPC_unknown: llvm_unreachable("Clause is not allowed."); } |