Skip to content

Commit 6bc000c

Browse files
committed
[SDAG] Remove else after return. NFC.
1 parent 6a6d6e5 commit 6bc000c

File tree

1 file changed

+20
-16
lines changed

1 file changed

+20
-16
lines changed

llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

Lines changed: 20 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -13898,18 +13898,20 @@ SDValue DAGCombiner::visitSIGN_EXTEND(SDNode *N) {
1389813898
// Op is i32, Mid is i8, and Dest is i32. If Op has more than 24 sign
1389913899
// bits, it is already ready.
1390013900
return Op;
13901-
} else if (OpBits < DestBits) {
13901+
}
13902+
13903+
if (OpBits < DestBits) {
1390213904
// Op is i32, Mid is i8, and Dest is i64. If Op has more than 24 sign
1390313905
// bits, just sext from i32.
1390413906
return DAG.getNode(ISD::SIGN_EXTEND, DL, VT, Op);
13905-
} else {
13906-
// Op is i64, Mid is i8, and Dest is i32. If Op has more than 56 sign
13907-
// bits, just truncate to i32.
13908-
SDNodeFlags Flags;
13909-
Flags.setNoSignedWrap(true);
13910-
Flags.setNoUnsignedWrap(N0->getFlags().hasNoUnsignedWrap());
13911-
return DAG.getNode(ISD::TRUNCATE, DL, VT, Op, Flags);
1391213907
}
13908+
13909+
// Op is i64, Mid is i8, and Dest is i32. If Op has more than 56 sign
13910+
// bits, just truncate to i32.
13911+
SDNodeFlags Flags;
13912+
Flags.setNoSignedWrap(true);
13913+
Flags.setNoUnsignedWrap(N0->getFlags().hasNoUnsignedWrap());
13914+
return DAG.getNode(ISD::TRUNCATE, DL, VT, Op, Flags);
1391313915
}
1391413916

1391513917
// fold (sext (truncate x)) -> (sextinreg x).
@@ -14187,19 +14189,21 @@ SDValue DAGCombiner::visitZERO_EXTEND(SDNode *N) {
1418714189
// Op is i32, Mid is i8, and Dest is i32. If Op has more than 24 sign
1418814190
// bits, it is already ready.
1418914191
return Op;
14190-
} else if (OpBits < DestBits) {
14192+
}
14193+
14194+
if (OpBits < DestBits) {
1419114195
// Op is i32, Mid is i8, and Dest is i64. If Op has more than 24 sign
1419214196
// bits, just sext from i32.
1419314197
// FIXME: This can probably be ZERO_EXTEND nneg?
1419414198
return DAG.getNode(ISD::SIGN_EXTEND, DL, VT, Op);
14195-
} else {
14196-
// Op is i64, Mid is i8, and Dest is i32. If Op has more than 56 sign
14197-
// bits, just truncate to i32.
14198-
SDNodeFlags Flags;
14199-
Flags.setNoSignedWrap(true);
14200-
Flags.setNoUnsignedWrap(true);
14201-
return DAG.getNode(ISD::TRUNCATE, DL, VT, Op, Flags);
1420214199
}
14200+
14201+
// Op is i64, Mid is i8, and Dest is i32. If Op has more than 56 sign
14202+
// bits, just truncate to i32.
14203+
SDNodeFlags Flags;
14204+
Flags.setNoSignedWrap(true);
14205+
Flags.setNoUnsignedWrap(true);
14206+
return DAG.getNode(ISD::TRUNCATE, DL, VT, Op, Flags);
1420314207
}
1420414208
}
1420514209

0 commit comments

Comments
 (0)