diff options
author | Kaelyn Takata <rikka@google.com> | 2014-12-02 22:05:35 +0000 |
---|---|---|
committer | Kaelyn Takata <rikka@google.com> | 2014-12-02 22:05:35 +0000 |
commit | c71dda2c38db839bca7262fd3a91a3982c720ddd (patch) | |
tree | debd9c84e59b4bd6cfb6fe989afa37cf941cf9b7 /clang/lib/Parse/ParseExpr.cpp | |
parent | bbdee936388d96bc4c72a3b635ad5d29ff6b6eac (diff) | |
download | bcm5719-llvm-c71dda2c38db839bca7262fd3a91a3982c720ddd.tar.gz bcm5719-llvm-c71dda2c38db839bca7262fd3a91a3982c720ddd.zip |
Diagnose TypoExprs in a couple of error cases in ParsePostfixExpressionSuffix.
Also have CorrectDelayedTyposInExpr check that the Expr* isn't null
before trying to access its members. Fixes PR21679.
llvm-svn: 223162
Diffstat (limited to 'clang/lib/Parse/ParseExpr.cpp')
-rw-r--r-- | clang/lib/Parse/ParseExpr.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/clang/lib/Parse/ParseExpr.cpp b/clang/lib/Parse/ParseExpr.cpp index 43929c0bcf2..eeea15dec05 100644 --- a/clang/lib/Parse/ParseExpr.cpp +++ b/clang/lib/Parse/ParseExpr.cpp @@ -1364,8 +1364,10 @@ Parser::ParsePostfixExpressionSuffix(ExprResult LHS) { if (!LHS.isInvalid() && !Idx.isInvalid() && Tok.is(tok::r_square)) { LHS = Actions.ActOnArraySubscriptExpr(getCurScope(), LHS.get(), Loc, Idx.get(), RLoc); - } else + } else { + (void)Actions.CorrectDelayedTyposInExpr(LHS); LHS = ExprError(); + } // Match the ']'. T.consumeClose(); @@ -1536,8 +1538,10 @@ Parser::ParsePostfixExpressionSuffix(ExprResult LHS) { /*AllowDestructorName=*/true, /*AllowConstructorName=*/ getLangOpts().MicrosoftExt, - ObjectType, TemplateKWLoc, Name)) + ObjectType, TemplateKWLoc, Name)) { + (void)Actions.CorrectDelayedTyposInExpr(LHS); LHS = ExprError(); + } if (!LHS.isInvalid()) LHS = Actions.ActOnMemberAccessExpr(getCurScope(), LHS.get(), OpLoc, |