Re: [PATCH] ia32: Fix alignment of _Atomic fields [PR65146]

2020-08-27 Thread H.J. Lu via Gcc-patches
On Thu, Aug 27, 2020 at 5:20 AM Uros Bizjak wrote: > > On Thu, Aug 27, 2020 at 10:35 AM Jakub Jelinek wrote: > > > > Hi! > > > > For _Atomic fields, lowering the alignment of long long or double etc. > > fields on ia32 is undesirable, because then one really can't perform atomic > > operations on

Re: [PATCH] ia32: Fix alignment of _Atomic fields [PR65146]

2020-08-27 Thread Uros Bizjak via Gcc-patches
On Thu, Aug 27, 2020 at 10:35 AM Jakub Jelinek wrote: > > Hi! > > For _Atomic fields, lowering the alignment of long long or double etc. > fields on ia32 is undesirable, because then one really can't perform atomic > operations on those using cmpxchg8b. > > The following patch stops lowering the a

[PATCH] ia32: Fix alignment of _Atomic fields [PR65146]

2020-08-27 Thread Jakub Jelinek via Gcc-patches
Hi! For _Atomic fields, lowering the alignment of long long or double etc. fields on ia32 is undesirable, because then one really can't perform atomic operations on those using cmpxchg8b. The following patch stops lowering the alignment in fields for _Atomic types (the x86_field_alignment change)