Author: Jeffrey Byrnes Date: 2022-10-14T07:56:45-07:00 New Revision: 30fc9fa3a4695f99b0aaabcec2e05118e8ee4b61
URL: https://github.com/llvm/llvm-project/commit/30fc9fa3a4695f99b0aaabcec2e05118e8ee4b61 DIFF: https://github.com/llvm/llvm-project/commit/30fc9fa3a4695f99b0aaabcec2e05118e8ee4b61.diff LOG: cleaned up print statements, checking load/store behavior Added: Modified: llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp llvm/lib/CodeGen/TargetLoweringBase.cpp Removed: ################################################################################ diff --git a/llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp b/llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp index 081d8d96c9e2..fe358aa89881 100644 --- a/llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp @@ -958,10 +958,6 @@ void SelectionDAGLegalize::LegalizeLoadOps(SDNode *Node) { /// Return a legal replacement for the given operation, with all legal operands. void SelectionDAGLegalize::LegalizeOp(SDNode *Node) { - errs() << "\n\n\nDAG BEFORE\n"; - DAG.dump(); - errs() << "\n"; - LLVM_DEBUG(dbgs() << "\nLegalizing: "; Node->dump(&DAG)); // Allow illegal target nodes and illegal registers. @@ -970,35 +966,14 @@ void SelectionDAGLegalize::LegalizeOp(SDNode *Node) { return; #ifndef NDEBUG -/* + for (unsigned i = 0, e = Node->getNumValues(); i != e; ++i) { - errs() << "Checking legality of: \n"; - auto temp = Node->getOperand(i-1); - temp.dump(); - errs() << "\n"; assert(TLI.getTypeAction(*DAG.getContext(), Node->getValueType(i)) == TargetLowering::TypeLegal && "Unexpected illegal type!"); } -*/ - errs() << "Quick Legal Check\n"; - for (const SDValue &Op : Node->op_values()) { - Op.dump(); - errs() << "\n"; - assert(TLI.getTypeAction(*DAG.getContext(), Op.getValueType()) == - TargetLowering::TypeLegal && - "Unexpected illegal type!"); - } - - errs() << "Full Legal Check\n"; for (const SDValue &Op : Node->op_values()) { - errs() << "Checking op: \n"; - Op.dump(); - errs() << "\n"; - if (TLI.getTypeAction(*DAG.getContext(), Op.getValueType()) == - TargetLowering::TypeLegal) errs() << "TargetLowering::TypeLegal\n"; - if (Op.getOpcode() == ISD::Register) errs() << "Register\n"; assert((TLI.getTypeAction(*DAG.getContext(), Op.getValueType()) == TargetLowering::TypeLegal || Op.getOpcode() == ISD::TargetConstant || @@ -1314,12 +1289,10 @@ void SelectionDAGLegalize::LegalizeOp(SDNode *Node) { LLVM_DEBUG(dbgs() << "Legal node: nothing to do\n"); return; case TargetLowering::Custom: - errs() << "from legalizeDAG.cpp\n"; LLVM_DEBUG(dbgs() << "Trying custom legalization\n"); // FIXME: The handling for custom lowering with multiple results is // a complete mess. if (SDValue Res = TLI.LowerOperation(SDValue(Node, 0), DAG)) { - errs() << "TLI.LowerOperation returned\n"; if (!(Res.getNode() != Node || Res.getResNo() != 0)) return; diff --git a/llvm/lib/CodeGen/TargetLoweringBase.cpp b/llvm/lib/CodeGen/TargetLoweringBase.cpp index 363bafb48c55..e6b577b4cc68 100644 --- a/llvm/lib/CodeGen/TargetLoweringBase.cpp +++ b/llvm/lib/CodeGen/TargetLoweringBase.cpp @@ -954,16 +954,13 @@ void TargetLoweringBase::setJumpIsExpensive(bool isExpensive) { TargetLoweringBase::LegalizeKind TargetLoweringBase::getTypeConversion(LLVMContext &Context, EVT VT) const { - errs() << "in TLB::getTypeConv\n"; // If this is a simple type, use the ComputeRegisterProp mechanism. if (VT.isSimple()) { - errs() << "isSimple]\n"; MVT SVT = VT.getSimpleVT(); assert((unsigned)SVT.SimpleTy < std::size(TransformToType)); MVT NVT = TransformToType[SVT.SimpleTy]; - errs() << "Found TypeTransform" << (int)NVT.SimpleTy << "\n"; LegalizeTypeAction LA = ValueTypeActions.getTypeAction(SVT); - errs() << "Found TypeAction: " << (int)LA << "\n"; + assert((LA == TypeLegal || LA == TypeSoftenFloat || LA == TypeSoftPromoteHalf || _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
