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.
cheers,
DaveK
