From: Eli Friedman <eli.friedman@gmail.com>
Date: Wed, 17 Dec 2008 22:19:57 +0000 (+0000)
Subject: Do proper recovery from an invalid switch condiition.  Fixes PR3229.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2342ef75797a2ad6c9d7a784cfff220fd1a66008;p=clang

Do proper recovery from an invalid switch condiition.  Fixes PR3229.



git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@61160 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Parse/ParseStmt.cpp b/lib/Parse/ParseStmt.cpp
index 9f8771c919..d22fbb7044 100644
--- a/lib/Parse/ParseStmt.cpp
+++ b/lib/Parse/ParseStmt.cpp
@@ -611,9 +611,10 @@ Parser::OwningStmtResult Parser::ParseSwitchStatement() {
   OwningExprResult Cond(Actions);
   if (ParseParenExprOrCondition(Cond))
     return StmtError();
-  
-  OwningStmtResult Switch(Actions,
-                          Actions.ActOnStartOfSwitchStmt(Cond.release()));
+
+  OwningStmtResult Switch(Actions);
+  if (!Cond.isInvalid())
+    Switch = Actions.ActOnStartOfSwitchStmt(Cond.release());
 
   // C99 6.8.4p3 - In C99, the body of the switch statement is a scope, even if
   // there is no compound stmt.  C90 does not have this clause.  We only do this
diff --git a/test/Parser/recovery.c b/test/Parser/recovery.c
index 78addb90e4..ea59516041 100644
--- a/test/Parser/recovery.c
+++ b/test/Parser/recovery.c
@@ -65,3 +65,6 @@ int test6248081() {
   [10]  // expected-error {{expected expression}}
 }
 
+struct forward;
+void x(struct forward* x) {switch(x->a) {}}
+