Re: [patch i386 mingw g++.dg gcc.dg]: Set -mms-bitfields as default for native windows targets

[Kai Tietz <ktietz70@googlemail.com>]

2011/4/13 Pedro Alves <pedro@codesourcery.com>:
> On Tuesday 12 April 2011 19:53:42, Kai Tietz wrote:
>> 2011/4/12 Richard Henderson <rth@redhat.com>:
>> > On 04/12/2011 05:47 AM, Kai Tietz wrote:
>> >> ChangeLog gcc/
>> >>
>> >>       * config/i386/mingw32.h (TARGET_SUBTARGET_DEFAULT): Add
>> >>       MASK_MS_BITFIELD_LAYOUT bit.
>> >>
>> >> ChangeLog gcc/testsuite
>> >>
>> >>       * g++.dg/ext/bitfield2.C: Add for i?86/x86_64-*-mingw*
>> >>       targets the additional -mno-ms-bitfields and
>> >>       -Wno-packed-bitfield-compat options.
>> >>       * g++.dg/ext/bitfield3.C: Likewise.
>> >>       * g++.dg/ext/bitfield4.C: Likewise.
>> >>       * g++.dg/ext/bitfield5.C: Likewise.
>> >>       * gcc.dg/bitfld-15.c: Likewise.
>> >>       * gcc.dg/bitfld-16.c: Likewise.
>> >>       * gcc.dg/bitfld-17.c: Likewise.
>> >>       * gcc.dg/bitfld-18.c: Likewise.
>> >
>> > Ok.
>> >
>> >
>> > r~
>> >
>>
>>
>> Applied at revision 172336.
>>
>> Thanks,
>> Kai
>>
>
> This is a default ABI change (IIRC, when the option was
> introduced, it was left off as default so to not break the ABI).
>
> Shouldn't we advertise it somewhere?
>
> --
> Pedro Alves
>

Yes, I did recently a lot of abi changing for mingw's targets.  They
all will need a nice description (and possible ways to get old default
behavior) in changes.html

I collect them for later.  Or do you mean it should be noted earlier?

Regards,
Kai