http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51200

--- Comment #2 from Joey Ye <joey.ye at arm dot com> 2011-11-22 03:58:29 UTC ---
Here is a test case fix.

With this patch, backend part of Bernd's original patch can be skipped. Thus
DJ's concern of unnecessary change can be addressed.

Also this test case intends to warn a situation that is incompatible to abi
version 1, -fstrict-volatile-bitfields happenly hides the incompatibility. IMHO
it is consultable to claim strict volatile bitfields violates version 1. So
fixing the test case and make it work as intended is more reasonable to me.


--- a/gcc/testsuite/g++.dg/abi/bitfield12.C
+++ b/gcc/testsuite/g++.dg/abi/bitfield12.C
@@ -1,4 +1,4 @@
-// { dg-options "-Wabi -fabi-version=1" }
+// { dg-options "-Wabi -fabi-version=1 -fno-strict-volatile-bitfields" }

 struct S { // { dg-warning "ABI" }
   char c : 1024; // { dg-warning "width" }

 struct S { // { dg-warning "ABI" }
   char c : 1024; // { dg-warning

Reply via email to