On Dec 16, 2011, at 9:56 AM, Dave Korn <dave.korn.cyg...@gmail.com> wrote:
> On 16/12/2011 09:01, Kai Tietz wrote:
>> 2011/12/15 Dave Korn:
>
>>> { dg-options "-mno-align-double" { target i?86-*-cygwin* i?86-*-mingw* } }
>>> { dg-additional-options "-mno-ms-bitfields" { target i?86-*-mingw* } }
>>>
>>> ... so that MinGW gets both and Cygwin only the one it wants? (Actually the
>>> first one could just as well be changed to dg-additional-options at the same
>>> time, couldn't it?)
>>
>> Well, interesting. I think it should be the additional variant for
>> cygwin/mingw, as otherwise -O2 gets clobbered for it, isn't it?
>
> Yes, that's what I was concerned with.
>
>> So I modified patch as attached.
>
> Thanks for that. I recommend this patch for approval.
Ok.
