[PATCH] D14797: [BranchFolding] Add volatile checking when clearing memory references

Geoff Berry via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 18 18:25:10 PST 2015


gberry added a subscriber: gberry.
gberry added a comment.

Hi Junmo,

Instead of adding a field to every MachineInstr, it might be better to concatenate the two memref lists in the case that they are unequal when being merged.
You could try to do something more complex like sorting the merged list and removing duplicates, but that might be overkill depending on how large these memref lists can get.


http://reviews.llvm.org/D14797





More information about the llvm-commits mailing list