From c5b32bd0c56ad71da2e6f35a6283041900a3535c Mon Sep 17 00:00:00 2001 From: Erik Pilkington <erik.pilkington@gmail.com> Date: Mon, 13 Jun 2016 20:56:45 +0000 Subject: [PATCH] [Parser] Only correct delayed typos when needed ActOnBinOp corrects delayed typos when in C mode; don't correct them in that case. Fixes PR26700. Differential Revision: http://reviews.llvm.org/D20490 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@272587 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Parse/ParseExpr.cpp | 4 ++++ test/Sema/typo-correction.c | 8 ++++++++ 2 files changed, 12 insertions(+) diff --git a/lib/Parse/ParseExpr.cpp b/lib/Parse/ParseExpr.cpp index 927bb208c75..4e39d42ea6a 100644 --- a/lib/Parse/ParseExpr.cpp +++ b/lib/Parse/ParseExpr.cpp @@ -446,6 +446,10 @@ Parser::ParseRHSOfBinaryExpression(ExprResult LHS, prec::Level MinPrec) { LHS = Actions.ActOnBinOp(getCurScope(), OpToken.getLocation(), OpToken.getKind(), LHS.get(), RHS.get()); + + // In this case, ActOnBinOp performed the CorrectDelayedTyposInExpr check. + if (!getLangOpts().CPlusPlus) + continue; } else { LHS = Actions.ActOnConditionalOp(OpToken.getLocation(), ColonLoc, LHS.get(), TernaryMiddle.get(), diff --git a/test/Sema/typo-correction.c b/test/Sema/typo-correction.c index 53b3b67ef2b..a1107a25ee5 100644 --- a/test/Sema/typo-correction.c +++ b/test/Sema/typo-correction.c @@ -57,3 +57,11 @@ void fn2() { } int d = X ? d : L; // expected-error 2 {{use of undeclared identifier}} + +int fn_with_ids() { ID = ID == ID >= ID ; } // expected-error 4 {{use of undeclared identifier}} + +int fn_with_rs(int r) { r = TYPO + r * TYPO; } // expected-error 2 {{use of undeclared identifier}} + +void fn_with_unknown(int a, int b) { + fn_with_unknown(unknown, unknown | unknown); // expected-error 3 {{use of undeclared identifier}} +} -- GitLab