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

[Momchil Velikov via Phabricator via cfe-commits]

chill added inline comments.


================
Comment at: lib/Parse/ParseStmt.cpp:843-845
+  return ParseCompoundStatement(isStmtExpr,
+                                Scope::DeclScope | Scope::CompoundStmtScope);
 }
----------------
rsmith wrote:
> 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.)
Yes, all the other cases are not applicable to C90, they are C++, ObjC, etc and the new flag is tested only in one place and only for C90.
However, for the sake of consistency, I've updated the patch to set the compound statement flag on all compound statements, including function bodies, try/catch/finally/blocks and whatnot.



https://reviews.llvm.org/D33676