This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH][RFA][LRA] Don't try to break down subreg expressions if insn already matches
- From: Kyrill Tkachov <kyrylo dot tkachov at arm dot com>
- To: Jeff Law <law at redhat dot com>, Maxim Kuvyrkov <maxim dot kuvyrkov at linaro dot org>
- Cc: Andrew Pinski <pinskia at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Vladimir Makarov <vmakarov at redhat dot com>
- Date: Tue, 17 Feb 2015 09:54:43 +0000
- Subject: Re: [PATCH][RFA][LRA] Don't try to break down subreg expressions if insn already matches
- Authentication-results: sourceware.org; auth=none
- References: <54DDC859 dot 3010805 at arm dot com> <41018A63-9F81-4798-9E72-ADAA723D75FD at gmail dot com> <54DDD624 dot 50406 at arm dot com> <8F75E52A-2ED8-4404-90FA-E5FF921E4F87 at linaro dot org> <54E2DD39 dot 3020500 at redhat dot com>
On 17/02/15 06:18, Jeff Law wrote:
On 02/14/15 04:23, Maxim Kuvyrkov wrote:
FYI, (and not related to the core issue of this patch)
The use of mult vs shift by combine is a problem that Venkat is
working on, see "[RFC] Tighten memory type assumption in RTL combiner
pass" . The combiner uses MULTs instead of SHIFTs for rtx'es that
look like addresses, even when they are, in fact, mere
Right. I think I put that into my gcc-6 queue because we don't have a
regression that requires this problem be fixed.
If you've got a BZ that's regressing because of this issue, don't
hesitate to point it out and I'll move the thread into my gcc-5 queue.
Right, so I think I'll wait for GCC-6 when this work is done to try
implement these patterns with shifts.
This LRA patch breaks on aarch64 anyway.