On Mon, Mar 30, 2015 at 04:02:36PM +0200, Jakub Jelinek wrote:
> On Mon, Mar 30, 2015 at 03:00:54PM +0100, James Greenhalgh wrote:
> > Likewise on ARM. The testcase is fairly obviously not going to work
> > for any target with 32-bit long:
> > 
> >   struct S
> >   {
> >     long l: 1;
> >     long l2: 41;
> >     unsigned long ul: 1;
> >     unsigned long ul2: 41;
> >   } s;
> > 
> >   .../c-c++-common/pr65556.c:9:3: error: width of 'ul2' exceeds its type
> 
> Patch to change s/long/long long/ preapproved, if it misbehaves the same
> with Marek's patch reverted.

Or I could just remove the :41 bit-fields from the testcase altogether, they
weren't needed to trigger the bug anyway.

        Marek

Reply via email to