This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch i386]: Fix PR/60193
- From: Kai Tietz <ktietz70 at googlemail dot com>
- To: Richard Henderson <rth at redhat dot com>
- Cc: Uros Bizjak <ubizjak at gmail dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Eric Botcazou <ebotcazou at adacore dot com>
- Date: Fri, 14 Feb 2014 16:23:12 +0100
- Subject: Re: [patch i386]: Fix PR/60193
- Authentication-results: sourceware.org; auth=none
- References: <CAFULd4ZxpvCJDHsH2Hdm6AZ1yJktg8HwV0BFC2atPxJLgR8hmA at mail dot gmail dot com> <CAEwic4aFeZNZ3WTm+vhXnLuM8s1xXuu3u3FRiM5hueePwe_N7Q at mail dot gmail dot com> <CAEwic4Y62439EB8y5Ea-sFDZcoLt7JdSozXp=H364AWvkTzMzw at mail dot gmail dot com> <52FE3422 dot 4080208 at redhat dot com>
2014-02-14 16:20 GMT+01:00 Richard Henderson <rth@redhat.com>:
> On 02/14/2014 06:41 AM, Kai Tietz wrote:
>> + else
>> + {
>> + t = plus_constant (Pmode, eax, UNITS_PER_WORD);
>> + emit_move_insn (eax, t);
>> + t = gen_rtx_PLUS (Pmode, stack_pointer_rtx, eax);
>> + }
>
> Uros is right that you don't need the move here: 8(rsp,rax) is a perfectly fine
> address.
>
>
> r~
Oh, right. I missed that. Is prior patch ok with that adjustment?
Regards,
Kai