On Fri, Aug 14, 2009 at 1:51 PM, Ted Kremenek<[email protected]> wrote: > --- cfe/trunk/lib/Sema/SemaDeclAttr.cpp (original) > +++ cfe/trunk/lib/Sema/SemaDeclAttr.cpp Fri Aug 14 15:51:58 2009 > @@ -438,13 +438,14 @@ > return; > } > > - if (!isFunction(d)) { > + const FunctionDecl *FD = dyn_cast<FunctionDecl>(d); > + > + if (!FD) { > S.Diag(Attr.getLoc(), diag::warn_attribute_wrong_decl_type) > << Attr.getName() << 0 /*function*/; > return; > }
Unless I'm mistaken, this breaks constructs like the following: __attribute((malloc)) void *(*f)(); -Eli _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
