diff options
| author | Alexey Bataev <a.bataev@hotmail.com> | 2016-04-01 08:43:42 +0000 |
|---|---|---|
| committer | Alexey Bataev <a.bataev@hotmail.com> | 2016-04-01 08:43:42 +0000 |
| commit | c5970623e0acf3d436f1c629caf0b5f3131e0ad5 (patch) | |
| tree | 0da685ebc291681778a602f220052c525325e824 /clang/lib | |
| parent | 215d59e7b0b14968c2f0c33c6575373455a84359 (diff) | |
| download | bcm5719-llvm-c5970623e0acf3d436f1c629caf0b5f3131e0ad5.tar.gz bcm5719-llvm-c5970623e0acf3d436f1c629caf0b5f3131e0ad5.zip | |
Revert "[OPENMP] Allow skip expression after comma in clauses with lists."
This reverts commit http://reviews.llvm.org/rL265003. After some
thoughts decided to emit errors here.
llvm-svn: 265119
Diffstat (limited to 'clang/lib')
| -rw-r--r-- | clang/lib/Parse/ParseOpenMP.cpp | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/clang/lib/Parse/ParseOpenMP.cpp b/clang/lib/Parse/ParseOpenMP.cpp index 68e725c0c7d..96cf37912f2 100644 --- a/clang/lib/Parse/ParseOpenMP.cpp +++ b/clang/lib/Parse/ParseOpenMP.cpp @@ -1414,18 +1414,17 @@ OMPClause *Parser::ParseOpenMPVarListClause(OpenMPDirectiveKind DKind, // Parse variable ExprResult VarExpr = Actions.CorrectDelayedTyposInExpr(ParseAssignmentExpression()); - if (VarExpr.isUsable()) + if (VarExpr.isUsable()) { Vars.push_back(VarExpr.get()); - else { + } else { SkipUntil(tok::comma, tok::r_paren, tok::annot_pragma_openmp_end, StopBeforeMatch); } // Skip ',' if any IsComma = Tok.is(tok::comma); - if (IsComma) { + if (IsComma) ConsumeToken(); - IsComma = false; - } else if (Tok.isNot(tok::r_paren) && + else if (Tok.isNot(tok::r_paren) && Tok.isNot(tok::annot_pragma_openmp_end) && (!MayHaveTail || Tok.isNot(tok::colon))) Diag(Tok, diag::err_omp_expected_punc) |
