diff options
| author | Eli Friedman <efriedma@codeaurora.org> | 2018-06-08 21:16:56 +0000 |
|---|---|---|
| committer | Eli Friedman <efriedma@codeaurora.org> | 2018-06-08 21:16:56 +0000 |
| commit | 864df22307ba16694828a1fc7b0b318498731dbf (patch) | |
| tree | 8d3f78d669801c3b4d3a29d7bcf765353b449dd8 /llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp | |
| parent | 79510be7cfd297e04b37d4def233aa416cdbc8f4 (diff) | |
| download | bcm5719-llvm-864df22307ba16694828a1fc7b0b318498731dbf.tar.gz bcm5719-llvm-864df22307ba16694828a1fc7b0b318498731dbf.zip | |
[ARM] Allow CMPZ transforms even if the input has multiple uses.
It looks like this got left in by accident in r289794; I can't think of
any reason this check would be necessary. (Maybe it was meant to be a
check that the AND has one use? But we check that a few lines earlier.)
Differential Revision: https://reviews.llvm.org/D47921
llvm-svn: 334322
Diffstat (limited to 'llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp')
| -rw-r--r-- | llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp b/llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp index fa41f880e3a..1a0ffe4e176 100644 --- a/llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp +++ b/llvm/lib/Target/ARM/ARMISelDAGToDAG.cpp @@ -2459,7 +2459,7 @@ void ARMDAGToDAGISel::SelectCMPZ(SDNode *N, bool &SwitchEQNEToPLMI) { SDValue X = And.getOperand(0); auto C = dyn_cast<ConstantSDNode>(And.getOperand(1)); - if (!C || !X->hasOneUse()) + if (!C) return; auto Range = getContiguousRangeOfSetBits(C->getAPIntValue()); if (!Range) |
