Re: RFA: Fix bogus mode in choose_reload_regs

2011-07-07 Thread Jay Foad
On 7 July 2011 09:09, Richard Sandiford wrote: > gcc/ >        * reload1.c (choose_reload_regs): Use mode sizes to check whether >        an old relaod register completely defines the required value. s/relaod/reload/ Jay.

Re: RFA: Fix bogus mode in choose_reload_regs

2011-07-07 Thread Ulrich Weigand
Richard Sandiford wrote: > gcc/ > * reload1.c (choose_reload_regs): Use mode sizes to check whether > an old relaod register completely defines the required value. > > gcc/testsuite/ > * gcc.target/arm/neon-modes-3.c: New test. This is OK. Thanks, Ulrich -- Dr. Ulrich Weig

RFA: Fix bogus mode in choose_reload_regs

2011-07-07 Thread Richard Sandiford
This patch fixes an ICE in smallest_mode_for_size on the attached testcase. The smallest_mode_for_size call comes from this part of the reload inheritance code in choose_reload_regs: if (byte == 0) need_mode = mode; else n