This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [rs6000] Fix ICE with -fstack-limit-register and large frames
- From: Eric Botcazou <ebotcazou at adacore dot com>
- To: Segher Boessenkool <segher at kernel dot crashing dot org>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Fri, 02 Jun 2017 10:27:33 +0200
- Subject: Re: [rs6000] Fix ICE with -fstack-limit-register and large frames
- Authentication-results: sourceware.org; auth=none
- References: <1944532.Kj3BqOocuy@polaris> <20170526131836.GE19687@gate.crashing.org>
> Could you make the expander handle it, instead? It's as simple as (after
> the double-reg thing) add "if operands[1] is reg 0, force_reg operands[2]".
> I'll do it if you prefer.
Probably, because I'm not sure how this can work, as you cannot create new
pseudos here.
> [ the patch is broken here ]
It applies just fine for me though. But it could probably use add_operand
instead of satisfies_constraint_I in the condition.
> > /* { dg-do compile } */
> > /* { dg-options "-fstack-limit-register=r2" } */
>
> Please use a different register, r2 already has different functions in
> most ABIs. It *probably* will compile anyway, but :-)
It's a straight copy of gcc.target/powerpc/pr48344-1.c though.
--
Eric Botcazou