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