[PATCH] Improve combiner's find_split_point (PR target/54589)
Sat Dec 1 00:31:00 GMT 2018
On Thu, Nov 29, 2018 at 04:36:01PM -0600, Segher Boessenkool wrote:
> Hi Jakub,
> On Thu, Nov 29, 2018 at 10:49:21PM +0100, Jakub Jelinek wrote:
> > The following patch attempts to improve find_split_point inside of
> > complex MEM addresses, if the target supports REG + REG + const
> > addressing, but doesn't support more_complex_rtx + REG + const,
> > try to split it at more_complex_rtx rather than more_complex_rtx + REG.
> > 2018-11-29 Jakub Jelinek <email@example.com>
> > PR target/54589
> > * combine.c (find_split_point): For invalid memory address
> > nonobj + obj + const, if reg + obj + const is valid addressing
> > mode, split at nonobj. Use if rather than else if for the
> > fallback. Comment fixes.
> > * gcc.target/i386/pr54589.c: New test.
> That looks good, but let me try it on some bigger builds first.
Whoops, forgot to get back to you. I tested it, and it looks fine, it
optimises the code quite often :-) Please commit it to trunk.
More information about the Gcc-patches