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