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]

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


The following patch fixes:

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58785

LRA chooses constraint 'm' for const_int operand. It means that the const_int should be placed in memory but it does not happen as preferred reload class hook returns LO_REGS for class NO_REGS which is result of LRA choosing 'm'. I don't know why reload pass needs such value but it should be return NO_REGS IMHO as it results in much less reload insns.

Is this patch ok to commit to the trunk?

2013-10-30  Vladimir Makarov  <vmakarov@redhat.com>

        PR target/58785
        * config/arm/arm.c (arm_preferred_reload_class): Don't return
        LO_REGS for NO_REGS for LRA.

2013-10-30  Vladimir Makarov  <vmakarov@redhat.com>

        PR target/58785
        * gcc.target/arm/pr58785.c: New.


Attachment: pr58785.patch
Description: Text document


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