From 22af55f8303c198ac610090e8a9c395bf17a3110 Mon Sep 17 00:00:00 2001 From: Matt Arsenault Date: Wed, 24 Jul 2024 15:53:01 +0400 Subject: [PATCH] FastISel: Do not check for module debug info I don't see the point of this check and SelectionDAG does not perform it. In the normal usecase, if there's no debug info the debug intrinsics would not be there in the first place. --- llvm/lib/CodeGen/SelectionDAG/FastISel.cpp | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/llvm/lib/CodeGen/SelectionDAG/FastISel.cpp b/llvm/lib/CodeGen/SelectionDAG/FastISel.cpp index ef9f783355190..e255bbaa92b16 100644 --- a/llvm/lib/CodeGen/SelectionDAG/FastISel.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/FastISel.cpp @@ -1203,11 +1203,6 @@ void FastISel::handleDbgInfo(const Instruction *II) { if (DbgLabelRecord *DLR = dyn_cast(&DR)) { assert(DLR->getLabel() && "Missing label"); - if (!FuncInfo.MF->getMMI().hasDebugInfo()) { - LLVM_DEBUG(dbgs() << "Dropping debug info for " << *DLR << "\n"); - continue; - } - BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, DLR->getDebugLoc(), TII.get(TargetOpcode::DBG_LABEL)) .addMetadata(DLR->getLabel()); @@ -1402,12 +1397,6 @@ bool FastISel::selectIntrinsicCall(const IntrinsicInst *II) { case Intrinsic::dbg_declare: { const DbgDeclareInst *DI = cast(II); assert(DI->getVariable() && "Missing variable"); - if (!FuncInfo.MF->getMMI().hasDebugInfo()) { - LLVM_DEBUG(dbgs() << "Dropping debug info for " << *DI - << " (!hasDebugInfo)\n"); - return true; - } - if (FuncInfo.PreprocessedDbgDeclares.contains(DI)) return true; @@ -1446,11 +1435,6 @@ bool FastISel::selectIntrinsicCall(const IntrinsicInst *II) { case Intrinsic::dbg_label: { const DbgLabelInst *DI = cast(II); assert(DI->getLabel() && "Missing label"); - if (!FuncInfo.MF->getMMI().hasDebugInfo()) { - LLVM_DEBUG(dbgs() << "Dropping debug info for " << *DI << "\n"); - return true; - } - BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, MIMD, TII.get(TargetOpcode::DBG_LABEL)).addMetadata(DI->getLabel()); return true;