[PATCH] D57954: [ARM] LoadStoreOptimizer: reoder limit
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 8 12:14:40 PST 2019
efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.
I guess this is fine. LGTM
Ultimately, the whole design here is sort of flaky: the scheduling code here isn't aware of register pressure, and the register allocator isn't aware of the possibility of generating ldm/stm, so the end result is very unreliable. I'm not sure what the right solution is, but tweaking the scheduling heuristics isn't going to help much.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57954/new/
https://reviews.llvm.org/D57954
More information about the llvm-commits
mailing list