This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] Remove DEFAULT_SHORT_ENUMS.


Hi Richard,

> > 	* opts.c (decode_options): Don't use DEFAULT_SHORT_ENUMS.
> > 	* system.h (DEFAULT_SHORT_ENUMS): Poison.
> > 	* config/cris/cris.h: Remove a comment about
> > 	DEFAULT_SHORT_ENUMS.
> > 	* config/ip2k/ip2k.h: Likewise.
> > 	* doc/tm.texi (DEFAULT_SHORT_ENUMS): Remove.
> > 
> > This is OK.
> 
> Please *don't* remove this code.  Short enums is the default in the ARM 
> EABI, so we will only have to put this all back again shortly.

Err, can I re-apply my patch?  I think this is an excellent
opportunity to hookize the macro because no port is using the macro,
so nobody has to go through every port to hookize things.  All we have
to do is touch target.h and target-def.h, and I am happy to do that
for you.

Kazu Hirata


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]