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

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added inline comments.


================
Comment at: lib/Parse/ParseCXXInlineMethods.cpp:521-522
   // to be re-used for method bodies as well.
-  ParseScope FnScope(this, Scope::FnScope|Scope::DeclScope);
+  ParseScope FnScope(this, Scope::FnScope|Scope::DeclScope|
+                           Scope::CompoundStmtScope);
   Actions.ActOnStartOfFunctionDef(getCurScope(), LM.D);
----------------
Did clang-format produce this? The lack of spaces between the `|` operators is surprising. Same comment applies elsewhere, so you might want to run clang-format over the patch if you've not already done so.


https://reviews.llvm.org/D33676