This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: patch to enable LRA for ppc
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>, Michael Meissner <meissner at linux dot vnet dot ibm dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, "Bergner, Peter" <bergner at vnet dot ibm dot com>
- Date: Tue, 22 Oct 2013 10:21:32 -0400
- Subject: Re: patch to enable LRA for ppc
- Authentication-results: sourceware.org; auth=none
- References: <524DDB71 dot 6040703 at redhat dot com> <CAGWvny=AmXxjALad=L0=Lgzp+4k9-SKXsOZNAcWcumcsOiViJw at mail dot gmail dot com> <526495E8 dot 70701 at redhat dot com> <20131021155144 dot GA19634 at ibm-tiger dot the-meissners dot org> <5265E5FF dot 8080102 at redhat dot com>
On Mon, Oct 21, 2013 at 10:42 PM, Vladimir Makarov <vmakarov@redhat.com> wrote:
>> I would say lets add -mlra, but make the default OFF for the time being.
>> We
>> can always switch the default later.
>
> Sure, if you know some LRA problems it should not be on default. Moreover,
> if we still have the problems when releasing gcc4.9, I think we should
> exclude any possibility for a user to use LRA for ppc. I don't want to have
> GGC-4.9 users blaming LRA.
>
> But adding LRA to PPC on the trunk (switched OFF by default) earlier could
> help me a lot to work on the issues.
My main concern was disrupting Mike. If Mike is comfortable with
adding LRA disabled by default, it is okay with me.
The patch mostly adds lra_in_progress, which will not have any effect
while LRA remains disabled.
My one question about the patch is:
- [(set (match_operand:DI 0 "reg_or_mem_operand" "=&r,Z,??&r")
+ [(set (match_operand:DI 0 "reg_or_mem_operand" "=&r,Z,&r")
which may cause register preferencing problems for bswap when LRA is not used.
The rest of the patch is okay.
Thanks, David