Skip to content

Commit 310d14e

Browse files
committed
Remove unnnecessary node duplication
1 parent eb851bf commit 310d14e

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -735,7 +735,7 @@ static bool hasSourceMods(const SDNode *N) {
735735
case ISD::INLINEASM:
736736
case ISD::INLINEASM_BR:
737737
case AMDGPUISD::DIV_SCALE:
738-
case ISD::INTRINSIC_W_CHAIN:
738+
case ISD::INTRINSIC_W_CHAIN:
739739

740740
// TODO: Should really be looking at the users of the bitcast. These are
741741
// problematic because bitcasts are used to legalize all stores to integer
@@ -4086,9 +4086,8 @@ SDValue AMDGPUTargetLowering::performShlCombine(SDNode *N,
40864086
LHSAND, Zero);
40874087
SDValue Hi =
40884088
DAG.getNode(ISD::EXTRACT_VECTOR_ELT, SL, MVT::i32, LHSAND, One);
4089-
SDValue AndMask = DAG.getConstant(0x1f, SL, MVT::i32);
4090-
SDValue LoAnd = DAG.getNode(ISD::AND, SL, MVT::i32, Lo, AndMask);
4091-
SDValue HiAnd = DAG.getNode(ISD::AND, SL, MVT::i32, Hi, AndMask);
4089+
SDValue LoAnd = DAG.getNode(ISD::AND, SL, MVT::i32, Lo, RHSAND->getOperand(0));
4090+
SDValue HiAnd = DAG.getNode(ISD::AND, SL, MVT::i32, Hi, RHSAND->getOperand(0));
40924091
SDValue Trunc = DAG.getNode(ISD::TRUNCATE, SL, MVT::i32, LHS);
40934092
if (AndIndex == 0 || AndIndex == 1)
40944093
return DAG.getNode(ISD::SHL, SL, MVT::i32, Trunc,

0 commit comments

Comments
 (0)