PATCH: Add -fno-short-enums to Symbian OS configuration

Mark Mitchell mark@codesourcery.com
Fri Aug 6 15:55:00 GMT 2004


Paul Brook wrote:

>>2004-08-06  Mark Mitchell  <mark@codesourcery.com>
>>
>>	* config/arm/symbian.h (CC1_SPEC): Add -fno-short-enums.
>>	(CC1PLUS_SPEC): Define it to be the same as CC1_SPEC.
>>    
>>
>
>Wouldn't this be better done using the TARGET_DEFAULT_SHORT_ENUMS target hook?
>  
>
To do that, we'd have to introduce a SUBTARGET_DEFAULT_SHORT_ENUMS or a 
flag in arm.c to indicate Symbian OS.  I'd been trying to avoid the latter.

>Failing that how about "%{!fshort-enums:-fno-short-enums}" ?
>  
>
Yes, that's better.  It's equivalent (since the user flag will always 
come later on the command-line) but it would be tidier.

Let me know what you and Richard E. think about what's cleanest here, 
and I'll tidy up.  I'll also use the same spec trick for the current 
-fno-builtin and -fvisibility settings.

Thanks,

-- 
Mark Mitchell
CodeSourcery, LLC
(916) 791-8304
mark@codesourcery.com



More information about the Gcc-patches mailing list