[PATCH] D33676: Place implictly declared functions at block scope

Richard Smith - zygoloid via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jul 9 06:14:17 PDT 2017


rsmith added inline comments.


================
Comment at: lib/Parse/ParseStmt.cpp:843-845
+  return ParseCompoundStatement(isStmtExpr,
+                                Scope::DeclScope | Scope::CompoundStmtScope);
 }
----------------
This seems to miss quite a lot of places that introduce compound statement scopes. (Search for callers of `ParseCompundStatementBody` and callers of the 2-argument form of `ParseCompoundStatement` for those.)


https://reviews.llvm.org/D33676





More information about the cfe-commits mailing list