[PATCH] Select Elimination in InstCombine

hfinkel at anl.gov hfinkel at anl.gov
Tue Sep 30 15:08:35 PDT 2014


================
Comment at: lib/Transforms/InstCombine/InstCombineCompares.cpp:2452
@@ +2451,3 @@
+    auto *Usr = cast<Instruction>(U);
+    if (Usr != UI && !DT->dominates(DB, Usr->getParent()))
+      return false;
----------------
If you mean for this to rule-out any other uses in the same block as the definition (I don't see anything else that might enforce that there is only one use in the parent block), should this be a call to properlyDominates?

http://reviews.llvm.org/D5258






More information about the llvm-commits mailing list