[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