diff options
Diffstat (limited to 'clang')
-rw-r--r-- | clang/lib/Parse/ParseExpr.cpp | 16 | ||||
-rw-r--r-- | clang/test/SemaTemplate/cxx1z-fold-expressions.cpp | 5 |
2 files changed, 13 insertions, 8 deletions
diff --git a/clang/lib/Parse/ParseExpr.cpp b/clang/lib/Parse/ParseExpr.cpp index 6913de9fa19..f4eef811755 100644 --- a/clang/lib/Parse/ParseExpr.cpp +++ b/clang/lib/Parse/ParseExpr.cpp @@ -2203,24 +2203,30 @@ Parser::ParseParenExpression(ParenParseOption &ExprType, bool stopIfCastExpr, Diag(Tok, diag::err_expected_lbrace_in_compound_literal); return ExprError(); } + } else if (Tok.is(tok::ellipsis) && + isFoldOperator(NextToken().getKind())) { + return ParseFoldExpression(ExprResult(), T); } else if (isTypeCast) { // Parse the expression-list. InMessageExpressionRAIIObject InMessage(*this, false); - + ExprVector ArgExprs; CommaLocsTy CommaLocs; if (!ParseSimpleExpressionList(ArgExprs, CommaLocs)) { + // FIXME: If we ever support comma expressions as operands to + // fold-expressions, we'll need to allow multiple ArgExprs here. + if (ArgExprs.size() == 1 && isFoldOperator(Tok.getKind()) && + NextToken().is(tok::ellipsis)) + return ParseFoldExpression(Result, T); + ExprType = SimpleExpr; Result = Actions.ActOnParenListExpr(OpenLoc, Tok.getLocation(), ArgExprs); } - } else if (Tok.is(tok::ellipsis) && - isFoldOperator(NextToken().getKind())) { - return ParseFoldExpression(ExprResult(), T); } else { InMessageExpressionRAIIObject InMessage(*this, false); - + Result = ParseExpression(MaybeTypeCast); ExprType = SimpleExpr; diff --git a/clang/test/SemaTemplate/cxx1z-fold-expressions.cpp b/clang/test/SemaTemplate/cxx1z-fold-expressions.cpp index 3934f011a35..8bb79113fa9 100644 --- a/clang/test/SemaTemplate/cxx1z-fold-expressions.cpp +++ b/clang/test/SemaTemplate/cxx1z-fold-expressions.cpp @@ -55,10 +55,9 @@ template<int ...N> void empty_with_base() { extern int k; (k = ... = N); // expected-warning{{unused}} - // FIXME: We misparse these. The first one looks a lot loke a declaration; - // it's not clear what's happening in the second one. void (k = ... = N); // expected-error {{expected ')'}} expected-note {{to match}} - (void) (k = ... = N); // expected-error {{expected ')'}} expected-note {{to match}} + void ((k = ... = N)); + (void) (k = ... = N); } template void empty_with_base<>(); // expected-note {{in instantiation of}} template void empty_with_base<1>(); |