llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: disservin (Disservin) <details> <summary>Changes</summary> Missed two other cases, so a follow up to https://github.com/llvm/llvm-project/pull/207085 --- Full diff: https://github.com/llvm/llvm-project/pull/207503.diff 1 Files Affected: - (modified) clang/lib/Parse/ParseStmt.cpp (+3-3) ``````````diff diff --git a/clang/lib/Parse/ParseStmt.cpp b/clang/lib/Parse/ParseStmt.cpp index f9c7d0988b403..53737f16f3869 100644 --- a/clang/lib/Parse/ParseStmt.cpp +++ b/clang/lib/Parse/ParseStmt.cpp @@ -1795,7 +1795,7 @@ StmtResult Parser::ParseDoStatement(LabelDecl *PrecedingLabel) { // C99 6.8.5p5 - In C99, the do statement is a block. This is not // the case for C90. Start the loop scope. - unsigned ScopeFlags = getLangOpts().C99 ? Scope::DeclScope : 0; + unsigned ScopeFlags = getLangOpts().C99 ? Scope::DeclScope : Scope::NoScope; ParseScope DoScope(this, ScopeFlags); // OpenACC Restricts a do-while-loop inside of certain construct/clause @@ -1922,8 +1922,8 @@ StmtResult Parser::ParseForStatement(SourceLocation *TrailingElseLoc, // doesn't cause declarations to bind to this scope. We use this to avoid // diagnosing a comma operator in e.g. the third part of a for loop when // '-Wcomma' is enabled. - unsigned ScopeFlags = - Scope::ControlScope | (C99orCXXorObjC ? Scope::DeclScope : 0); + unsigned ScopeFlags = Scope::ControlScope | + (C99orCXXorObjC ? Scope::DeclScope : Scope::NoScope); ParseScope ForScope(this, ScopeFlags); BalancedDelimiterTracker T(*this, tok::l_paren); `````````` </details> https://github.com/llvm/llvm-project/pull/207503 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
