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

Andrew Trick via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 23 10:37:39 PST 2015

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.



More information about the llvm-commits mailing list