[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