commit cc64bbca818f996ab43573b1c113d85bde15a420
author Dan Gohman <gohman@apple.com> Thu Apr 05 20:27:21 2012 +0000
committer Dan Gohman <gohman@apple.com> Thu Apr 05 20:27:21 2012 +0000
tree af170dfad46db1d37053dc3153b936e322caa9db
parent e8235fef3197e199ffd390eaf0e7cd039338c6a4

Fix accidentally inverted logic from r152803, and make the
testcase slightly less trivial. This fixes rdar://11171718.

llvm-svn: 154118

llvm/lib/Transforms/Scalar/ObjCARC.cpp

diff --git a/llvm/lib/Transforms/Scalar/ObjCARC.cpp b/llvm/lib/Transforms/Scalar/ObjCARC.cpp
index 80c1dd1..56e6c4c 100644
--- a/llvm/lib/Transforms/Scalar/ObjCARC.cpp
+++ b/llvm/lib/Transforms/Scalar/ObjCARC.cpp
@@ -4073,7 +4073,7 @@
         if (!UserInst)
           continue;
         // FIXME: dominates should return true for unreachable UserInst.
-        if (!DT->isReachableFromEntry(UserInst->getParent()) ||
+        if (DT->isReachableFromEntry(UserInst->getParent()) &&
             DT->dominates(Inst, UserInst)) {
           Changed = true;
           Instruction *Replacement = Inst;