[AST] fail rather than crash when const evaluating invalid c++ foreach

[hokein]

clang/test/SemaCXX/constexpr-function-recovery-crash.cpp

73

this can be simplified further.

struct X {} array[] = {undef()};

clang/lib/AST/ExprConstant.cpp

case Stmt::CXXForRangeStmtClass: {
     // Initialize the __range variable.
     EvalStmtResult ESR = EvaluateStmt(Result, Info, FS->getRangeStmt());
     if (ESR != ESR_Succeeded) {
       if (ESR != ESR_Failed && !Scope.destroy())
         return ESR_Failed;
       return ESR;
     }
 
+    // In error-recovery cases it's possible to get here even if we failed to
+    // synthesize the __begin and __end variables.
+    if (!FS->getBeginStmt() || !FS->getEndStmt() || !FS->getCond())
+      return ESR_Failed;
+
     // Create the __begin and __end iterators.
     ESR = EvaluateStmt(Result, Info, FS->getBeginStmt());
     if (ESR != ESR_Succeeded) {
       if (ESR != ESR_Failed && !Scope.destroy())
         return ESR_Failed;
       return ESR;
     }
     ESR = EvaluateStmt(Result, Info, FS->getEndStmt());

clang/test/SemaCXX/constexpr-function-recovery-crash.cpp

template<int x> constexpr int f(int y) { // expected-note {{candidate template ignored}}
   return x * y;
 }
 constexpr int test9(int x) {
   return f<1>(f<x>(1)); // expected-error {{no matching function for call to 'f'}}
 }
 
 constexpr int test10() { return undef(); } // expected-error {{use of undeclared identifier 'undef'}}
 static_assert(test10() <= 1, "should not crash"); // expected-error {{static_assert expression is not an integral constant expression}}
+
+struct X {} array[] = {undef()}; // expected-error {{use of undeclared identifier 'undef'}}
+constexpr void test11() {
+  for (X& e : array) {}
+}