commit 9d5849f51dcf662ae440b373089ce36c8d0f5757
author Benjamin Kramer <benny.kra@googlemail.com> Mon May 28 20:52:48 2012 +0000
committer Benjamin Kramer <benny.kra@googlemail.com> Mon May 28 20:52:48 2012 +0000
tree 5899d983cc86f4ace20eb696d5204025b695714a
parent 44a079aa2340887871897c70a404ff0523a6e84b

Fix suspicous hasOneUse() check, found by PVS Studio (PR12357).

llvm-svn: 157592

llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp

diff --git a/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp b/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
index 665b3c6..0f10d73 100644
--- a/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
+++ b/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp
@@ -2217,7 +2217,7 @@
   if (Op0I && Op1I && Op0I->isShift() && 
       Op0I->getOpcode() == Op1I->getOpcode() && 
       Op0I->getOperand(1) == Op1I->getOperand(1) &&
-      (Op1I->hasOneUse() || Op1I->hasOneUse())) {
+      (Op0I->hasOneUse() || Op1I->hasOneUse())) {
     Value *NewOp =
       Builder->CreateXor(Op0I->getOperand(0), Op1I->getOperand(0),
                          Op0I->getName());