diff options
author | Jakub Jelinek <jakub@redhat.com> | 2021-06-16 12:17:55 +0200 |
---|---|---|
committer | Jakub Jelinek <jakub@redhat.com> | 2021-06-16 12:17:55 +0200 |
commit | b4b50bf2864e09f028a39a3f460222632c4d7348 (patch) | |
tree | 158ad7b885e22418c2dac6cabb0002f47d740772 /libcpp | |
parent | 43fc4234ad3d9302d3460385b6fdb5e3f59b6986 (diff) | |
download | gcc-b4b50bf2864e09f028a39a3f460222632c4d7348.zip gcc-b4b50bf2864e09f028a39a3f460222632c4d7348.tar.gz gcc-b4b50bf2864e09f028a39a3f460222632c4d7348.tar.bz2 |
stor-layout: Create DECL_BIT_FIELD_REPRESENTATIVE even for bitfields in unions [PR101062]
The following testcase is miscompiled on x86_64-linux, the bitfield store
is implemented as a RMW 64-bit operation at d+24 when the d variable has
size of only 28 bytes and scheduling moves in between the R and W part
a store to a different variable that happens to be right after the d
variable.
The reason for this is that we weren't creating
DECL_BIT_FIELD_REPRESENTATIVEs for bitfields in unions.
The following patch does create them, but treats all such bitfields as if
they were in a structure where the particular bitfield is the only field.
2021-06-16 Jakub Jelinek <jakub@redhat.com>
PR middle-end/101062
* stor-layout.c (finish_bitfield_representative): For fields in unions
assume nextf is always NULL.
(finish_bitfield_layout): Compute bit field representatives also in
unions, but handle it as if each bitfield was the only field in the
aggregate.
* gcc.dg/pr101062.c: New test.
Diffstat (limited to 'libcpp')
0 files changed, 0 insertions, 0 deletions