On Tue, Sep 25, 2012 at 5:32 PM, Dinar Temirbulatov
<dtemirbula...@gmail.com> wrote:
> Hi Ramana,
> Here is obvious fix for PR49423, I just added pool range for

Sorry for the late response - I've been on vacation.

No it's not ok. These were removed deliberately  and subsequent
efforts to put these back on have been rejected as being incomplete.
Please investigate why reload wants to reload the load of a constant 0
which is a valid constant for HImode values into the constant pool .
In addition it might be that movhi_insn needs to grow support for movw
at the right architecture level.

With your patch and for example 920928-2.c you'll start seeing 0 going
into the constant pool and an ldrh out of that !

regards
Ramana

Reply via email to