This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: patch to fix PR59787 (arm target)
- From: Richard Earnshaw <rearnsha at arm dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>, yvan dot roux at linaro dot org
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 14 Jan 2014 18:07:54 +0000
- Subject: Re: RFA: patch to fix PR59787 (arm target)
- Authentication-results: sourceware.org; auth=none
- References: <52D56A56 dot 1060908 at redhat dot com>
On 14/01/14 16:48, Vladimir Makarov wrote:
> The following patch fixes
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59787
>
> The problem of LRA looping was in that address with soft frame pointer
> was rejected as valid constraint in iwmmxt_amd_movdi insn.
>
> Ok to commit?
>
> 2014-01-14 Vladimir Makarov <vmakarov@redhat.com>
>
> PR target/59787
> * config/arm/arm.c (arm_coproc_mem_operand): Add lra_in_progress.
A quick grep of the arm backend shows 11 instances of reload_in_progress:
arm.c: && !(reload_in_progress || reload_completed)
arm.c: if (! (reload_in_progress || reload_completed)
arm.c: if (! (reload_in_progress || reload_completed)
arm.c: if (! (reload_in_progress || reload_completed)
arm.c: reload_in_progress || reload_completed))
arm.md: "TARGET_32BIT && (reload_in_progress || reload_completed)"
arm.md: "TARGET_32BIT && (reload_in_progress || reload_completed)"
arm.md: "TARGET_32BIT && (reload_in_progress || reload_completed)"
arm.md: "TARGET_32BIT && (reload_in_progress || reload_completed)"
predicates.md: "offsettable_address_p (reload_completed |
reload_in_progress,
predicates.md: (and (match_test "reload_in_progress ||
reload_completed")
and aarch64 has five more:
aarch64.md: "reload_completed || reload_in_progress"
aarch64.md: "reload_completed || reload_in_progress"
aarch64.md: "reload_completed || reload_in_progress"
aarch64.md: "reload_completed || reload_in_progress"
aarch64.md: "reload_completed || reload_in_progress"
Yvan, could you do a quick audit on these to see if they are also likely
to need fixing?
R.