Re: [ARM] Fix minipool handling of aligned labels

2011-08-12 Thread Richard Sandiford
Steven Bosscher writes: > On Thu, Aug 11, 2011 at 1:56 PM, Richard Sandiford > wrote: > >> gcc/ >>        * config/arm/arm.c (get_label_padding): New function. >>        (create_fix_barrier, md_reorg): Likewise. > > The ChangeLog doesn't match the patch: > * create_fix_barrier is not a new functi

Re: [ARM] Fix minipool handling of aligned labels

2011-08-12 Thread Steven Bosscher
On Thu, Aug 11, 2011 at 1:56 PM, Richard Sandiford wrote: > gcc/ >        * config/arm/arm.c (get_label_padding): New function. >        (create_fix_barrier, md_reorg): Likewise. The ChangeLog doesn't match the patch: * create_fix_barrier is not a new function. * md_reorg should be arm_reorg Ci

Re: [ARM] Fix minipool handling of aligned labels

2011-08-11 Thread Richard Earnshaw
On 11/08/11 12:56, Richard Sandiford wrote: > Trying to compile a certain embedded benchmark with -falign-loops=8 > or above fails due to out-of-range references to the literal pool. > This is because arm_reorg doesn't take label alignment into account. > > Fixed with the patch below. Tested on a

[ARM] Fix minipool handling of aligned labels

2011-08-11 Thread Richard Sandiford
Trying to compile a certain embedded benchmark with -falign-loops=8 or above fails due to out-of-range references to the literal pool. This is because arm_reorg doesn't take label alignment into account. Fixed with the patch below. Tested on arm-linux-gnueabi. I also tried compiling libav with a