Re: [patch, ARM] PR48250, rehaul arm_legitimize_reload_address()

2011-04-20 Thread Richard Earnshaw
On Wed, 2011-04-20 at 23:06 +0800, Chung-Lin Tang wrote: On 2011/4/20 09:24 PM, Richard Sandiford wrote: Hi Chung-Lin, I'm seeing an ICE with this patch, specifically; Chung-Lin Tang clt...@codesourcery.com writes: + if (coproc_p) + low = SIGN_MAG_LOW_ADDR_BITS (val, 10);

Re: [patch, ARM] PR48250, rehaul arm_legitimize_reload_address()

2011-04-20 Thread Chung-Lin Tang
On 2011/4/20 11:12 PM, Richard Earnshaw wrote: On Wed, 2011-04-20 at 23:06 +0800, Chung-Lin Tang wrote: On 2011/4/20 09:24 PM, Richard Sandiford wrote: Hi Chung-Lin, I'm seeing an ICE with this patch, specifically; Chung-Lin Tang clt...@codesourcery.com writes: + if (coproc_p) +

Re: [patch, ARM] PR48250, rehaul arm_legitimize_reload_address()

2011-04-11 Thread Richard Earnshaw
On Fri, 2011-04-08 at 15:52 +0800, Chung-Lin Tang wrote: Hi Richard, here's a new patch, with some quite significant changes to arm_legitimize_reload_address(). This time fully utilizing the sign-magnitude offset macro you gave in the last thread, and adapting your description into embedded