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

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 3 10:49:37 PST 2015


mcrosier added a comment.

In http://reviews.llvm.org/D14797#301681, @gberry wrote:

> 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.


This sounds very reasonable, IMO.


http://reviews.llvm.org/D14797





More information about the llvm-commits mailing list