On Mon, Mar 30, 2015 at 04:06:56PM +0200, Marek Polacek wrote:
> 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.
Works for me too. Or turn them into :21 or similar (long must be at least
32-bit).
Jakub
