[PATCH] D37611: [IfConversion] More simple, correct dead/kill liveness handling

Kyle Butt via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 8 10:53:52 PDT 2017


iteratee added a comment.

This looks fine overall to me.



================
Comment at: lib/CodeGen/MachineBasicBlock.cpp:449
+      for (unsigned LiveReg : LiveRegs)
+        IsLive = IsLive || TRI->regsOverlap(Reg, LiveReg);
+
----------------
Would it be worth it to have an early exit from the loop when IsLive is true?


https://reviews.llvm.org/D37611





More information about the llvm-commits mailing list