[C++0x] contiguous bitfields race implementation

Submitted by Aldy Hernandez on July 27, 2011, 7:13 p.m.

Details

Message ID 4E306366.6010904@redhat.com
State New
Headers show

Commit Message

Aldy Hernandez July 27, 2011, 7:13 p.m.
On 07/27/11 13:55, Jakub Jelinek wrote:
> On Wed, Jul 27, 2011 at 01:51:04PM -0500, Aldy Hernandez wrote:
>>> This caused:
>>>
>>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49875
>>
>> The assembler sequence on ia32 was a bit different.
>>
>> H.J.  Can you try this on your end?  If it fixes the problem, I will
>> commit as obvious.
>
> You could test it yourself on x86_64-linux too with
> make check -k RUNTESTFLAGS='--target_board=unix\{-m32,-m64\} dg.exp=cxxbit*'

Committed.
PR middle-end/49875
	* c-c++-common/cxxbitfields-4.c: Check for smaller than long
	moves.
	* c-c++-common/cxxbitfields-5.c: Same.

Patch hide | download patch | download mbox

Index: c-c++-common/cxxbitfields-4.c
===================================================================
--- c-c++-common/cxxbitfields-4.c	(revision 176824)
+++ c-c++-common/cxxbitfields-4.c	(working copy)
@@ -15,4 +15,4 @@  void update_c(struct bits *p, int val) 
     p -> c = val;
 }
 
-/* { dg-final { scan-assembler-not "movl" } } */
+/* { dg-final { scan-assembler "mov\[bw\]" } } */
Index: c-c++-common/cxxbitfields-5.c
===================================================================
--- c-c++-common/cxxbitfields-5.c	(revision 176824)
+++ c-c++-common/cxxbitfields-5.c	(working copy)
@@ -26,4 +26,4 @@  void foo()
   p -> c = 55;
 }
 
-/* { dg-final { scan-assembler-not "movl\t\\(" } } */
+/* { dg-final { scan-assembler "mov\[bw\]" } } */