Re: [PATCH] ARM: Introduce ARM_DEFAULT_SHORT_ENUMS

2017-04-04 Thread Sebastian Huber
On 04/04/17 12:50, Sebastian Huber wrote: On 04/04/17 12:41, Ramana Radhakrishnan wrote: [dropping devel at rtems dot org as I don't want more bounces] On Tue, Apr 4, 2017 at 10:07 AM, Sebastian Huber wrote: >On 04/04/17 11:00, Ramana Radhakrishnan wrote: >>> >>> > static bool >>> > arm

Re: [PATCH] ARM: Introduce ARM_DEFAULT_SHORT_ENUMS

2017-04-04 Thread Sebastian Huber
On 04/04/17 12:41, Ramana Radhakrishnan wrote: [dropping devel at rtems dot org as I don't want more bounces] On Tue, Apr 4, 2017 at 10:07 AM, Sebastian Huber wrote: >On 04/04/17 11:00, Ramana Radhakrishnan wrote: >>> >>> > static bool >>> > arm_default_short_enums (void) >>> > { >>> >-

Re: [PATCH] ARM: Introduce ARM_DEFAULT_SHORT_ENUMS

2017-04-04 Thread Ramana Radhakrishnan
[dropping devel at rtems dot org as I don't want more bounces] On Tue, Apr 4, 2017 at 10:07 AM, Sebastian Huber wrote: > On 04/04/17 11:00, Ramana Radhakrishnan wrote: >>> >>> > static bool >>> > arm_default_short_enums (void) >>> > { >>> >- return TARGET_AAPCS_BASED && arm_abi != ARM_ABI_AAP

Re: [PATCH] ARM: Introduce ARM_DEFAULT_SHORT_ENUMS

2017-04-04 Thread Sebastian Huber
On 04/04/17 11:00, Ramana Radhakrishnan wrote: > static bool > arm_default_short_enums (void) > { >- return TARGET_AAPCS_BASED && arm_abi != ARM_ABI_AAPCS_LINUX; >+ return ARM_DEFAULT_SHORT_ENUMS; > } > > >diff --git a/gcc/config/arm/rtems.h b/gcc/config/arm/rtems.h >index 53cd987..b34bbe8

Re: [PATCH] ARM: Introduce ARM_DEFAULT_SHORT_ENUMS

2017-04-04 Thread Ramana Radhakrishnan
On Mon, Apr 3, 2017 at 12:32 PM, Sebastian Huber wrote: > Allow targets to define the default for the short enums option. > > gcc/ > > * config/arm/arm.c: (ARM_DEFAULT_SHORT_ENUMS): Provide default > definition. > * config/arm/rtems.h (ARM_DEFAULT_SHORT_ENUMS) Define. > ---

Re: [PATCH] ARM: Introduce ARM_DEFAULT_SHORT_ENUMS

2017-04-04 Thread Ramana Radhakrishnan
On Tue, Apr 4, 2017 at 8:43 AM, Bernhard Reutner-Fischer wrote: > On 3 April 2017 13:32:30 CEST, Sebastian Huber > wrote: >>Allow targets to define the default for the short enums option. > > Does this work nowadays? > About 10 years ago I did this for some arm and had to force TREE_CODE at al

Re: [PATCH] ARM: Introduce ARM_DEFAULT_SHORT_ENUMS

2017-04-04 Thread Sebastian Huber
On 04/04/17 09:43, Bernhard Reutner-Fischer wrote: On 3 April 2017 13:32:30 CEST, Sebastian Huber wrote: Allow targets to define the default for the short enums option. Does this work nowadays? About 10 years ago I did this for some arm and had to force TREE_CODE at al to 16 bit manually, II

Re: [PATCH] ARM: Introduce ARM_DEFAULT_SHORT_ENUMS

2017-04-04 Thread Bernhard Reutner-Fischer
On 3 April 2017 13:32:30 CEST, Sebastian Huber wrote: >Allow targets to define the default for the short enums option. Does this work nowadays? About 10 years ago I did this for some arm and had to force TREE_CODE at al to 16 bit manually, IIRC. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3420