[llvm] 8cc9c42 - [TargetLowering] Use LegalOnly operand to isOperationLegalOrCustom to simplify some code. NFC
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 4 12:31:50 PST 2021
Author: Craig Topper
Date: 2021-02-04T12:30:37-08:00
New Revision: 8cc9c42a0c260a0c3d7381e43f2e28dec957ec1f
URL: https://github.com/llvm/llvm-project/commit/8cc9c42a0c260a0c3d7381e43f2e28dec957ec1f
DIFF: https://github.com/llvm/llvm-project/commit/8cc9c42a0c260a0c3d7381e43f2e28dec957ec1f.diff
LOG: [TargetLowering] Use LegalOnly operand to isOperationLegalOrCustom to simplify some code. NFC
Added:
Modified:
llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp
Removed:
################################################################################
diff --git a/llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp b/llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp
index b801394c1a1e..dbf4919fff67 100644
--- a/llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp
+++ b/llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp
@@ -5158,11 +5158,9 @@ SDValue TargetLowering::BuildSDIV(SDNode *N, SelectionDAG &DAG,
// Multiply the numerator (operand 0) by the magic value.
// FIXME: We should support doing a MUL in a wider type.
SDValue Q;
- if (IsAfterLegalization ? isOperationLegal(ISD::MULHS, VT)
- : isOperationLegalOrCustom(ISD::MULHS, VT))
+ if (isOperationLegalOrCustom(ISD::MULHS, VT, IsAfterLegalization))
Q = DAG.getNode(ISD::MULHS, dl, VT, N0, MagicFactor);
- else if (IsAfterLegalization ? isOperationLegal(ISD::SMUL_LOHI, VT)
- : isOperationLegalOrCustom(ISD::SMUL_LOHI, VT)) {
+ else if (isOperationLegalOrCustom(ISD::SMUL_LOHI, VT, IsAfterLegalization)) {
SDValue LoHi =
DAG.getNode(ISD::SMUL_LOHI, dl, DAG.getVTList(VT, VT), N0, MagicFactor);
Q = SDValue(LoHi.getNode(), 1);
@@ -5286,11 +5284,9 @@ SDValue TargetLowering::BuildUDIV(SDNode *N, SelectionDAG &DAG,
// FIXME: We should support doing a MUL in a wider type.
auto GetMULHU = [&](SDValue X, SDValue Y) {
- if (IsAfterLegalization ? isOperationLegal(ISD::MULHU, VT)
- : isOperationLegalOrCustom(ISD::MULHU, VT))
+ if (isOperationLegalOrCustom(ISD::MULHU, VT, IsAfterLegalization))
return DAG.getNode(ISD::MULHU, dl, VT, X, Y);
- if (IsAfterLegalization ? isOperationLegal(ISD::UMUL_LOHI, VT)
- : isOperationLegalOrCustom(ISD::UMUL_LOHI, VT)) {
+ if (isOperationLegalOrCustom(ISD::UMUL_LOHI, VT, IsAfterLegalization)) {
SDValue LoHi =
DAG.getNode(ISD::UMUL_LOHI, dl, DAG.getVTList(VT, VT), X, Y);
return SDValue(LoHi.getNode(), 1);
More information about the llvm-commits
mailing list