On Wed, Feb 15, 2012 at 8:23 AM, Ye Joey <joey.ye...@gmail.com> wrote: > Ping^2
All hunks apart from the expr.c hunk from 182545 are ok (thus, all hunks that are explicitly guarded with -fstrict-volatile-bitfields). Please use -p for diffs, I couldn't spot the place in expr.c you patch, the line-numbers are off and the patch isn't against the 4.6 branch. Please state how you tested you backports. r182649 is ok (obvious ontop of 182545), likewise r182685. Richard. > On Mon, Jan 16, 2012 at 11:21 AM, Ye Joey <joey.ye...@gmail.com> wrote: >> Ping >> >> On Tue, Dec 27, 2011 at 10:19 AM, Ye Joey <joey.ye...@gmail.com> wrote: >>> Fix PR51200. Backport trunk 182545, 182649, 182685 to 4.6. >>> >>> OK to 4.6? >>> >>> - Joey >>> >>> 2011-12-20 Bernd Schmidt <ber...@codesourcery.com> >>> >>> PR middle-end/51200 >>> * expr.c (store_field): Avoid a direct store if the mode is larger >>> than the size of the bit field. >>> * stor-layout.c (layout_decl): If flag_strict_volatile_bitfields, >>> treat non-volatile bit fields like volatile ones. >>> * toplev.c (process_options): Disallow combination of >>> -fstrict-volatile-bitfields and ABI versions less than 2. >>> * config/arm/arm.c (arm_option_override): Don't enable >>> flag_strict_volatile_bitfields if the ABI version is less than 2. >>> * config/h8300/h8300.c (h8300_option_override): Likewise. >>> * config/rx/rx.c (rx_option_override): Likewise. >>> * config/m32c/m32c.c (m32c_option_override): Likewise. >>> * config/sh/sh.c (sh_option_override): Likewise. >>> >>> 2011-12-22 Joey Ye <joey...@arm.com> >>> >>> * toplev.c (process_options): Fix typo. >>> >>> testsute >>> 2011-12-20 Bernd Schmidt <ber...@codesourcery.com> >>> >>> PR middle-end/51200 >>> * gcc.target/arm/volatile-bitfields-4.c: New test. >>> * c-c++-common/abi-bf.c: New test. >>> >>> 2011-12-26 Joey Ye <joey...@arm.com> >>> >>> PR middle-end/51200 >>> * gcc.dg/volatile-bitfields-2.c: New test.