[PATCH] D26954: Remove unnecessary IDom check

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 19 12:59:30 PDT 2017


davide added inline comments.


================
Comment at: Sink.cpp:172
     BasicBlock *Candidate = (*I)->getBlock();
-    if ((*I)->getIDom()->getBlock() == Inst->getParent() &&
-        IsAcceptableTarget(Inst, Candidate, DT, LI))
+    if (IsAcceptableTarget(Inst, Candidate, DT, LI))
       SuccToSinkTo = Candidate;
----------------
Add a comment here to explain why you don't need to check.


Repository:
  rL LLVM

https://reviews.llvm.org/D26954





More information about the llvm-commits mailing list