diff mbox

[testsuite,g++.dg] : Reflect ABI change for windows native targets about bitfield layout in structures

Message ID CAEwic4ZpvA0oeeJP75ZUOH_8YBqvDfL9GKULbJDxwhPW+WO5vQ@mail.gmail.com
State New
Headers show

Commit Message

Kai Tietz Dec. 15, 2011, 10:09 a.m. UTC 
Hi,

In gcc 4.7 the default for IA mingw targets about ms-bitfields was
changed. So patch
adjusts testcase to make sure we disable it for mingw targets

ChangeLog

2011-12-15  Kai Tietz  <ktietz@redhat.com>

	* g++.dg/abi/bitfield3.C: Make sure test runs
	for windows targets without activated ms-bitfields option,

Ok for apply?
Kai

 {

Comments

Rainer Orth Dec. 15, 2011, 10:17 a.m. UTC | #1 
Kai Tietz <ktietz70@googlemail.com> writes:

> Index: g++.dg/abi/bitfield3.C
> ===================================================================
> --- g++.dg/abi/bitfield3.C	(revision 182092)
> +++ g++.dg/abi/bitfield3.C	(working copy)
> @@ -3,7 +3,7 @@
>  // { dg-options "-O2" }
>  // Cygwin and mingw32 default to MASK_ALIGN_DOUBLE. Override to ensure
>  // 4-byte alignment.
> -// { dg-options "-mno-align-double" { target i?86-*-cygwin* i?86-*-mingw* } }
> +// { dg-options "-mno-align-double -mno-ms-bitfields" { target
> i?86-*-cygwin* i?86-*-mingw* } }

The comment only explains the -mno-align-double now.  Would you please
add an explanation for -mno-ms-bitfields, too?

Thanks.
        Rainer
diff mbox

Patch

Index: g++.dg/abi/bitfield3.C
===================================================================
--- g++.dg/abi/bitfield3.C	(revision 182092)
+++ g++.dg/abi/bitfield3.C	(working copy)
@@ -3,7 +3,7 @@ 
 // { dg-options "-O2" }
 // Cygwin and mingw32 default to MASK_ALIGN_DOUBLE. Override to ensure
 // 4-byte alignment.
-// { dg-options "-mno-align-double" { target i?86-*-cygwin* i?86-*-mingw* } }
+// { dg-options "-mno-align-double -mno-ms-bitfields" { target
i?86-*-cygwin* i?86-*-mingw* } }

 struct A