This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: RFA: patch to fix PR58785 (an ARM LRA crash)


On 11/27/13 03:19, Yvan Roux wrote:
Ping.

On 20 November 2013 10:22, Yvan Roux <yvan.roux@linaro.org> wrote:
Hi,

as Richard said, only a subset of rclass is allowed to be returned by
preferred_reload_class.
I don't think he was quite that definitive. "One reading of the manual suggests ...". However, Richard's interpretation is the same as I've had for eons. You can return the original class, a narrower class or NO_REGS.





  I've tested the attached patched in Thumb
mode, on ARMv5, A9 and A9hf and on cross A15 without regression.

Yvan

2013-11-20  Yvan Roux  <yvan.roux@linaro.org>

         PR target/58785
         * config/arm/arm.c (arm_preferred_reload_class): Only return LO_REGS
         when rclass is GENERAL_REGS.
This is fine for the trunk as it implements Richard's recommendation and brings the code into compliance with the semantics of P_R_C.

Jeff


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]