[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