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

[Andrew Trick via llvm-commits]

atrick added a comment.

> To restate to make sure I understand.  When merging two instructions, it's always legal to concatenate (and possibly remove duplicates) the original two lists.  When splitting, it's correct - but slightly conservative - to keep the original list.  In either case, it is *legal* to use a new empty list, but not recommended due to imprecision.


That's my understanding, FWIW. We should try not to drop memoperands, but anticipate that some authors of target passes could be lazy.


Repository:
  rL LLVM

http://reviews.llvm.org/D15730