[PATCH] D15730: [MachineLICM] Fix handling of memoperands

Andrew Trick via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 22 19:07:30 PST 2015


atrick accepted this revision.
atrick added a comment.
This revision is now accepted and ready to land.

Thanks for clarifying the spec and fixing assumptions. It looks like there is a temptation to assume that access to a frame index must have non-empty memoperands. But that would be totally unenforceable. If there's a performance issue, it's probably better to fix cases where LLVM drops memoperands rather than make those assumptions.


http://reviews.llvm.org/D15730





More information about the llvm-commits mailing list