[PATCH] D16156: Proper handling of diamond-like cases in if-conversion

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 20 05:21:44 PST 2016


kparzysz marked an inline comment as done.

================
Comment at: lib/CodeGen/IfConversion.cpp:1492-1493
@@ +1491,4 @@
+  if (!BBI2->BB->empty() && (DI2 == BBI2->BB->end())) {
+    auto BBI1T = BBI1->BB->getFirstTerminator();
+    auto BBI2T = BBI2->BB->getFirstTerminator();
+    if ((BBI1T != BBI1->BB->end()) && TII->isPredicated(BBI1T) &&
----------------
MatzeB wrote:
> I'd prefer an explicit MachineBasicBlock::iterator here instead of auto.
Done.


Repository:
  rL LLVM

http://reviews.llvm.org/D16156





More information about the llvm-commits mailing list