system.h & warnings (patch for gansidecl.h included)
Kaveh R. Ghazi
ghazi@caip.rutgers.edu
Mon Apr 6 17:35:00 GMT 1998
> From: Jeffrey A Law <law@hurl.cygnus.com>
>
> In message < 199804031726.MAA22875@caip.rutgers.edu >you write:
>
> > It provides for centralized maintenance. I don't know if any
> > attributes were added after gcc-2.7, but the current ubiquitous test
> > assumes that all attributes work as of version 2.7. That may not hold
> > in the future and we should have a centralized way of handling these on
> > a per attribute basis. This ensures if we have to distinguish between
> > them for the purposes of figuring out which ones work as of some gcc
> > version, they are already separated in the source code and we don't have
> > to change more than one file.
> >
> > Below is the patch I came up with. Is it okay to install?
> I'm not convinced we need this kind of centralized maintenance for
> attributes, since I don't forsee that we'll use them all that often.
>
> However, I also don't see a good reason not to install your patch :-)
>
> So, go ahead and install it :-)
>
>
> > Fri Apr 3 09:50:55 1998 Kaveh R. Ghazi <ghazi@caip.rutgers.edu>
> >
> > * gansidecl.h: Check if compiler supports __attribute__. Provide
> > definitions for ATTRIBUTE_UNUSED and ATTRIBUTE_PRINTF using
> > __attribute__ when its available. Also provide definitions for
> > ATTRIBUTE_PRINTF_1, ATTRIBUTE_PRINTF_2 and ATTRIBUTE_PRINTF_3 in
> > terms of ATTRIBUTE_PRINTF.
> >
> > * genoutput.c (process_template): Use ATTRIBUTE_UNUSED in place
> > of __attribute__.
>
> jeff
Done.
--
Kaveh R. Ghazi Project Manager / Custom Development
ghazi@caip.rutgers.edu Icon CMT Corp.
More information about the Gcc
mailing list