On 04/15/2016 08:10 AM, Kyrill Tkachov wrote:
Hi all,

This is a repost of Andrew's fix for PR target/64971 that was originally
posted at:
https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00502.html

The only change is that I substituted DImode for Pmode and added a FIXME
comment
to remind us to revisit this (see the PR in bugzilla for more info).

Bootstrapped and tested on aarch64-none-linux-gnu (LP64, I don't have
access to a full ILP32 system)
This patch affects only ILP32 codegen so I've run a make check on
aarch64-none-elf with /-mabi=ilp32 and nothing regressed.
I think at this stage it's the least risky band-aid.

Is this ok for trunk at this stage?

Thanks,
Kyrill

2016-04-15  Andrew Pinski  <apin...@cavium.com>
             Kyrylo Tkachov  <kyrylo.tkac...@arm.com>

     PR target/65971
     * config/aarch64/aarch64.md (sibcall): Force call
     address to be DImode for ILP32.
     (sibcall_value): Likewise.

2016-04-15  Andrew Pinski  <apin...@cavium.com>

     * gcc.c-torture/compile/pr37433-1.c: New testcase.
Just a note, gcc-6 has branched, so you'll need to apply to the trunk and the branch once approved.

I would suggest that the issues raised by Richard be addressed separately, possibly using a new BZ for tracking purposes.

Jeff

Reply via email to