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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to