On Wed, Oct 22, 2014 at 8:13 PM, Richard Smith <[email protected]> wrote: > On Wed, Oct 22, 2014 at 2:44 PM, Aaron Ballman <[email protected]> > wrote: >> >> On Wed, Oct 22, 2014 at 5:21 PM, David Blaikie <[email protected]> wrote: >> > My only quandry is: what other things can the body be? If CompoundStmt >> > and >> > and CXXTryStmt are the only ones, then we can remove the conditional >> > entirely. >> >> I think we can remove that conditional. IssueWarnings calls getBody() >> on the Decl passed in from PopFunctionScopeInfo, and that should >> always be something function-like, and I can't think of a situation >> where we'd get something other than a CompoundStmt or CXXTryStmt. >> >> Richard, can you think of any situations that would be problematic? > > > No, the body of a function is always one of those two things.
Assuming I remove the isa<> tests, does the patch LGTY? ~Aaron _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
