diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp index 6ae88ce3d86a..008c65673ed2 100644 --- a/clang/lib/Sema/SemaDeclCXX.cpp +++ b/clang/lib/Sema/SemaDeclCXX.cpp @@ -2103,11 +2103,12 @@ CheckConstexprFunctionStmt(Sema &SemaRef, const FunctionDecl *Dcl, Stmt *S, break; if (!Cxx1yLoc.isValid()) Cxx1yLoc = S->getBeginLoc(); - for (Stmt *SubStmt : S->children()) + for (Stmt *SubStmt : S->children()) { if (SubStmt && !CheckConstexprFunctionStmt(SemaRef, Dcl, SubStmt, ReturnStmts, Cxx1yLoc, Cxx2aLoc, Cxx2bLoc, Kind)) return false; + } return true; case Stmt::SwitchStmtClass: @@ -2118,22 +2119,24 @@ CheckConstexprFunctionStmt(Sema &SemaRef, const FunctionDecl *Dcl, Stmt *S, // mutation, we can reasonably allow them in C++11 as an extension. if (!Cxx1yLoc.isValid()) Cxx1yLoc = S->getBeginLoc(); - for (Stmt *SubStmt : S->children()) + for (Stmt *SubStmt : S->children()) { if (SubStmt && !CheckConstexprFunctionStmt(SemaRef, Dcl, SubStmt, ReturnStmts, Cxx1yLoc, Cxx2aLoc, Cxx2bLoc, Kind)) return false; + } return true; case Stmt::LabelStmtClass: case Stmt::GotoStmtClass: if (Cxx2bLoc.isInvalid()) Cxx2bLoc = S->getBeginLoc(); - for (Stmt *SubStmt : S->children()) + for (Stmt *SubStmt : S->children()) { if (SubStmt && !CheckConstexprFunctionStmt(SemaRef, Dcl, SubStmt, ReturnStmts, Cxx1yLoc, Cxx2aLoc, Cxx2bLoc, Kind)) return false; + } return true; case Stmt::GCCAsmStmtClass: