[PATCH] D34099: [IfConversion] Maintain the CFG when predicating/merging blocks in IfConvert*

Kyle Butt via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 19 13:59:53 PDT 2017


iteratee added a comment.

Mostly looks good. It will be easier to follow when you factor out the NFC sections and rebase.



================
Comment at: lib/CodeGen/IfConversion.cpp:1487
     MergeBlocks(BBI, *CvtBBI);
   }
 
----------------
This section looks like an NFC. Please commit it first, and then rebase this patch on top of it.


================
Comment at: lib/CodeGen/IfConversion.cpp:1587
+
   if (CvtMBB.pred_size() > 1) {
     // Copy instructions in the true block, predicate them, and add them to
----------------
Same for this section. Please factor out the NFC change and commit it first.


https://reviews.llvm.org/D34099





More information about the llvm-commits mailing list