> On Oct 16, 2017, at 1:56 PM, Wilco Dijkstra <wilco.dijks...@arm.com> wrote: > > This patch cleans up autopref scheduling. > > The code is greatly simplified. Sort accesses on the offset first, and > only if the offsets are the same fall back to other comparisons in > rank_for_schedule. This doesn't at all restore the original behaviour > since we no longer compare the base address, but it now defines a total > sorting order. More work will be required to improve the sorting so > that only loads/stores with the same base are affected. > > AArch64 bootstrap completes. > > OK for commit?
Looks good to me. I'm not an official scheduler maintainer, so wait for an ack from a maintainer. -- Maxim Kuvyrkov www.linaro.org
