[PATCH] D14797: [BranchFolding] Merge MMOs during tail merge

Geoff Berry via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 3 10:39:00 PST 2015


gberry added a comment.

Regarding an MMO compare function, I don't think this would be that much different from the existing MachineMemOperand::operator==.
Alternatively, could you just add a bail out case that clears all memops if M1->getMemOperands().size() * M2->getMemOperands().size() is above a certain threshold?  That would prevent any pathological cases from causing compile time blowups.


http://reviews.llvm.org/D14797





More information about the llvm-commits mailing list