On 4/17/19 4:13 AM, Segher Boessenkool wrote: > When auto-inc-dec creates a new mem to compute the cost of doing some > transform, it forgets to copy over the alignment of the original mem. > This gives wrong costs, for example, for rs6000 a floating point load > or store is hugely expensive if unaligned. This patch fixes it. > > This doesn't fix any test case I'm aware of, but it is a very simple > patch. Is it okay for trunk? > > > Segher > > > 2019-04-17 Segher Boessenkool <seg...@kernel.crashing.org> > > * auto-inc-dec.c (attempt_change): Set the alignment of the > temporary memory to that of the original. Given this is only changing the RTL passed into the costing calculations, I think it can go in now.
OK for the trunk. jeff
