[PATCH] D126700: [MachineScheduler] Order more stores by ascending address

Allen zhong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 7 19:16:11 PDT 2022


Allen added a comment.

In D126700#3564241 <https://reviews.llvm.org/D126700#3564241>, @dmgreen wrote:

> OK I see, thanks. In this version on godbolt I only see ordered STPQ's, perhaps you have some downstream differences that would alter the codegen?
> So - do you have any other benchmark results, on any other cpus? Or as an alternative, do you think that adding a subtarget feature is an OK solution?

Many thanks @dmgreen for your detailed reply, and I fint the newest upstream already sort the stores by ascending address ,  I'll try to figure out it. https://godbolt.org/z/W4vqocjz3 (in the .LBB0_5)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126700/new/

https://reviews.llvm.org/D126700



More information about the llvm-commits mailing list