[cfe-commits] r79041 - /cfe/trunk/lib/Sema/SemaDeclAttr.cpp

[Eli Friedman]

On Fri, Aug 14, 2009 at 1:51 PM, Ted Kremenek<kremenek at apple.com> 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