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