Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-17 Thread Sandra Loosemore
Earlier today I wrote: On 06/17/2013 08:41 AM, Joseph S. Myers wrote: On Mon, 17 Jun 2013, Julian Brown wrote: IIUC, the incompatibility between the specified -fstrict-volatile-bitfields behaviour and the C++ memory model is a recognised deficiency in the ARM EABI. It might be an unpopular su

Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-17 Thread Sandra Loosemore
On 06/17/2013 08:41 AM, Joseph S. Myers wrote: On Mon, 17 Jun 2013, Julian Brown wrote: IIUC, the incompatibility between the specified -fstrict-volatile-bitfields behaviour and the C++ memory model is a recognised deficiency in the ARM EABI. It might be an unpopular suggestion, but how about d

Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-17 Thread Joseph S. Myers
On Mon, 17 Jun 2013, Julian Brown wrote: > IIUC, the incompatibility between the specified > -fstrict-volatile-bitfields behaviour and the C++ memory model is a > recognised deficiency in the ARM EABI. It might be an unpopular > suggestion, but how about disabling the option by default for C++ on

Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-17 Thread Richard Biener
On Mon, Jun 17, 2013 at 2:38 PM, Bernd Schmidt wrote: > On 06/17/2013 02:27 PM, Julian Brown wrote: >> On Mon, 17 Jun 2013 13:38:05 +0200 >> Richard Biener wrote: >> >>> On Mon, Jun 17, 2013 at 1:31 PM, Julian Brown >>> wrote: IIUC, the incompatibility between the specified -fstrict-vo

Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-17 Thread Bernd Schmidt
On 06/17/2013 02:27 PM, Julian Brown wrote: > On Mon, 17 Jun 2013 13:38:05 +0200 > Richard Biener wrote: > >> On Mon, Jun 17, 2013 at 1:31 PM, Julian Brown >> wrote: >>> IIUC, the incompatibility between the specified >>> -fstrict-volatile-bitfields behaviour and the C++ memory model is a >>> re

Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-17 Thread Jakub Jelinek
On Mon, Jun 17, 2013 at 01:27:38PM +0100, Julian Brown wrote: > Well -- I'm certainly no expert on the C++ memory model, but I am under > the impression (that I can't seem to verify by googling ;-)) that > accesses to adjacent bitfields during volatile access of a particular > bitfield are forbidde

Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-17 Thread Julian Brown
On Mon, 17 Jun 2013 13:38:05 +0200 Richard Biener wrote: > On Mon, Jun 17, 2013 at 1:31 PM, Julian Brown > wrote: > > On Mon, 17 Jun 2013 13:12:09 +0200 > > Richard Biener wrote: > > > >> On Sun, Jun 16, 2013 at 9:26 PM, Jakub Jelinek > >> wrote: > >> > On Sun, Jun 16, 2013 at 01:08:12PM -0600

Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-17 Thread Richard Biener
On Mon, Jun 17, 2013 at 1:31 PM, Julian Brown wrote: > On Mon, 17 Jun 2013 13:12:09 +0200 > Richard Biener wrote: > >> On Sun, Jun 16, 2013 at 9:26 PM, Jakub Jelinek >> wrote: >> > On Sun, Jun 16, 2013 at 01:08:12PM -0600, Sandra Loosemore wrote: >> >> This patch fixes the PR23623 regression. I

Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-17 Thread Julian Brown
On Mon, 17 Jun 2013 13:12:09 +0200 Richard Biener wrote: > On Sun, Jun 16, 2013 at 9:26 PM, Jakub Jelinek > wrote: > > On Sun, Jun 16, 2013 at 01:08:12PM -0600, Sandra Loosemore wrote: > >> This patch fixes the PR23623 regression. In conjunction with part > >> 2 of the series, it also fixes the

Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-17 Thread Richard Biener
On Sun, Jun 16, 2013 at 9:26 PM, Jakub Jelinek wrote: > On Sun, Jun 16, 2013 at 01:08:12PM -0600, Sandra Loosemore wrote: >> This patch fixes the PR23623 regression. In conjunction with part 2 >> of the series, it also fixes the new volatile-bitfields-3.c test >> case. >> >> As I noted in previou

Re: [patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-16 Thread Jakub Jelinek
On Sun, Jun 16, 2013 at 01:08:12PM -0600, Sandra Loosemore wrote: > This patch fixes the PR23623 regression. In conjunction with part 2 > of the series, it also fixes the new volatile-bitfields-3.c test > case. > > As I noted in previous discussion, there might be a better place to > accomplish t

[patch 3/5] don't restrict bit range for -fstrict-volatile-bitfields

2013-06-16 Thread Sandra Loosemore
This patch fixes the PR23623 regression. In conjunction with part 2 of the series, it also fixes the new volatile-bitfields-3.c test case. As I noted in previous discussion, there might be a better place to accomplish this effect, but hacking DECL_BIT_FIELD_REPRESENTATIVE can't work because t