diff options
| author | Benjamin Kramer <benny.kra@googlemail.com> | 2012-05-28 20:52:48 +0000 |
|---|---|---|
| committer | Benjamin Kramer <benny.kra@googlemail.com> | 2012-05-28 20:52:48 +0000 |
| commit | 9d5849f51dcf662ae440b373089ce36c8d0f5757 (patch) | |
| tree | 5899d983cc86f4ace20eb696d5204025b695714a /llvm/lib/Transforms | |
| parent | 44a079aa2340887871897c70a404ff0523a6e84b (diff) | |
| download | bcm5719-llvm-9d5849f51dcf662ae440b373089ce36c8d0f5757.tar.gz bcm5719-llvm-9d5849f51dcf662ae440b373089ce36c8d0f5757.zip | |
Fix suspicous hasOneUse() check, found by PVS Studio (PR12357).
llvm-svn: 157592
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp b/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp index 665b3c65a10..0f10d7376c3 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp +++ b/llvm/lib/Transforms/InstCombine/InstCombineAndOrXor.cpp @@ -2217,7 +2217,7 @@ Instruction *InstCombiner::visitXor(BinaryOperator &I) { 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()); |
