* gcc.dg/memmodel/subfields.c (set_a): Set noinline attribute. * g++.dg/memmodel/bitfields.C (set_a): Same. * g++.dg/memmodel/bitfields-2.C: New. Index: gcc.dg/memmodel/subfields.c =================================================================== --- gcc.dg/memmodel/subfields.c (revision 170937) +++ gcc.dg/memmodel/subfields.c (working copy) @@ -20,6 +20,7 @@ struct test_struct { not affect any of the other fields in the structure. An improper implementation may load an entire word, change the 8 bits for field 'a' and write the entire word back out. */ +__attribute__((noinline)) void set_a(char x) { var.a = x; Index: g++.dg/memmodel/bitfields-2.C =================================================================== --- g++.dg/memmodel/bitfields-2.C (revision 0) +++ g++.dg/memmodel/bitfields-2.C (revision 0) @@ -0,0 +1,71 @@ +/* { dg-do link } */ +/* { dg-options "-O2 --param allow-load-data-races=0 --param allow-store-data-races=0" } */ +/* { dg-final { memmodel-gdb-test } } */ + +/* Test that setting does not touch either or . + In the C++ memory model, non contiguous bitfields ("a" and "c" + here) should be considered as distinct memory locations, so we + can't use bit twiddling to set either one. */ + +#include +#include "memmodel.h" + +#define CONSTA 12 + +static int global; +struct S +{ + unsigned int a : 4; + unsigned char b; + unsigned int c : 6; +} var; + +__attribute__((noinline)) +void set_a() +{ + var.a = CONSTA; +} + +void memmodel_other_threads() +{ + ++global; + var.b = global; + var.c = global; +} + +int memmodel_step_verify() +{ + int ret = 0; + if (var.b != global) + { + printf ("FAIL: Unexpected value: var.b is %d, should be %d\n", + var.b, global); + ret = 1; + } + if (var.c != global) + { + printf ("FAIL: Unexpected value: var.c is %d, should be %d\n", + var.c, global); + ret = 1; + } + return ret; +} + +int memmodel_final_verify() +{ + int ret = memmodel_step_verify(); + if (var.a != CONSTA) + { + printf ("FAIL: Unexpected value: var.a is %d, should be %d\n", + var.a, CONSTA); + ret = 1; + } + return ret; +} + +int main() +{ + set_a(); + memmodel_done(); + return 0; +} Index: g++.dg/memmodel/bitfields.C =================================================================== --- g++.dg/memmodel/bitfields.C (revision 171248) +++ g++.dg/memmodel/bitfields.C (working copy) @@ -23,6 +23,7 @@ struct S unsigned int c : 6; } var; +__attribute__((noinline)) void set_a() { var.a = CONSTA;