[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.
More information about the llvm-commits