On Wed, Aug 12, 2009 at 7:49 PM, Eli Friedman<[email protected]> wrote: > On Wed, Aug 12, 2009 at 4:36 PM, Ryan Flynn<[email protected]> wrote: >> --- cfe/trunk/lib/Sema/SemaDeclAttr.cpp (original) >> +++ cfe/trunk/lib/Sema/SemaDeclAttr.cpp Wed Aug 12 18:36:28 2009 >> @@ -437,7 +437,7 @@ >> return; >> } >> >> - if (FunctionDecl *FD = dyn_cast<FunctionDecl>(d)) { >> + if (FunctionDecl *FD = cast<FunctionDecl>(d)) { >> if (!FD->getResultType()->isPointerType()) { >> S.Diag(Attr.getLoc(), diag::warn_attribute_malloc_pointer_only); >> return; > > Is there actually any guarantee the decl in question is a function decl? > > -Eli >
Eli, The block above it returns if (!isFunctionOrMethod(d)). Ryan _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
