[llvm] [NVPTX][DebugInfo] avoid emitting extra .loc directives (PR #84584)
Alex MacLean via llvm-commits
llvm-commits at lists.llvm.org
Sat Mar 16 10:34:23 PDT 2024
https://github.com/AlexMaclean updated https://github.com/llvm/llvm-project/pull/84584
>From 5e7068154485b9a50786a881845c0a04c39b94c6 Mon Sep 17 00:00:00 2001
From: Alex MacLean <amaclean at nvidia.com>
Date: Fri, 8 Mar 2024 22:31:44 +0000
Subject: [PATCH 1/2] [NVPTX][DebugInfo] avoid emitting extra .loc directives
---
llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp | 8 +++++--
llvm/test/DebugInfo/NVPTX/no-extra-loc.ll | 26 +++++++++++++++++++++++
2 files changed, 32 insertions(+), 2 deletions(-)
create mode 100644 llvm/test/DebugInfo/NVPTX/no-extra-loc.ll
diff --git a/llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp b/llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp
index 2219d9f6619aaf..ed5df37cbde2c7 100644
--- a/llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp
+++ b/llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp
@@ -489,8 +489,12 @@ void NVPTXAsmPrinter::emitFunctionEntryLabel() {
OutStreamer->emitRawText(StringRef("{\n"));
setAndEmitFunctionVirtualRegisters(*MF);
// Emit initial .loc debug directive for correct relocation symbol data.
- if (MMI && MMI->hasDebugInfo())
- emitInitialRawDwarfLocDirective(*MF);
+ const DISubprogram *SP = MF->getFunction().getSubprogram();
+ if (SP) {
+ assert(SP->getUnit());
+ if (!SP->getUnit()->isDebugDirectivesOnly() && MMI && MMI->hasDebugInfo())
+ emitInitialRawDwarfLocDirective(*MF);
+ }
}
bool NVPTXAsmPrinter::runOnMachineFunction(MachineFunction &F) {
diff --git a/llvm/test/DebugInfo/NVPTX/no-extra-loc.ll b/llvm/test/DebugInfo/NVPTX/no-extra-loc.ll
new file mode 100644
index 00000000000000..6d3a69beffe875
--- /dev/null
+++ b/llvm/test/DebugInfo/NVPTX/no-extra-loc.ll
@@ -0,0 +1,26 @@
+; RUN: llc < %s -mtriple=nvptx64-nvidia-cuda | FileCheck %s
+; RUN: %if ptxas %{ llc < %s -mtriple=nvptx64-nvidia-cuda | %ptxas-verify %}
+
+
+define i32 @foo(i32 %a, i32 %b) !dbg !3 {
+
+; CHECK: .loc [[FILE:[0-9]+]] 26 0 // extra-lineinfo.cu:26:0
+; CHECK-NOT: .loc [[FILE]] 26 0 // extra-lineinfo.cu:26:0
+; CHECK: .file [[FILE]] "/test/directory/extra-lineinfo.cu"
+
+ %add = add i32 %b, %a, !dbg !6
+ ret i32 %add, !dbg !6
+}
+
+!llvm.dbg.cu = !{!0}
+!nvvm.annotations = !{}
+!llvm.module.flags = !{!2}
+
+!0 = distinct !DICompileUnit(language: DW_LANG_C_plus_plus, file: !1, producer: "clang", isOptimized: true, runtimeVersion: 0, emissionKind: DebugDirectivesOnly)
+!1 = !DIFile(filename: "extra-lineinfo.cu", directory: "/test/directory/")
+!2 = !{i32 1, !"Debug Info Version", i32 3}
+!3 = distinct !DISubprogram(name: "kernel", linkageName: "foo", scope: !1, file: !1, line: 123, type: !4, scopeLine: 26, spFlags: DISPFlagDefinition | DISPFlagOptimized, unit: !0)
+!4 = !DISubroutineType(types: !5)
+!5 = !{}
+!6 = !DILocation(line: 40, column: 22, scope: !31)
+!31 = distinct !DILexicalBlock(scope: !3, file: !1, line: 3, column: 17)
>From 8e5500ba95e1a20c41a5265c78b4043ce6799de0 Mon Sep 17 00:00:00 2001
From: Alex MacLean <amaclean at nvidia.com>
Date: Sat, 16 Mar 2024 17:34:04 +0000
Subject: [PATCH 2/2] address comments
---
llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp b/llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp
index ed5df37cbde2c7..2783b1e0ec73eb 100644
--- a/llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp
+++ b/llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp
@@ -489,8 +489,7 @@ void NVPTXAsmPrinter::emitFunctionEntryLabel() {
OutStreamer->emitRawText(StringRef("{\n"));
setAndEmitFunctionVirtualRegisters(*MF);
// Emit initial .loc debug directive for correct relocation symbol data.
- const DISubprogram *SP = MF->getFunction().getSubprogram();
- if (SP) {
+ if (const DISubprogram *SP = MF->getFunction().getSubprogram()) {
assert(SP->getUnit());
if (!SP->getUnit()->isDebugDirectivesOnly() && MMI && MMI->hasDebugInfo())
emitInitialRawDwarfLocDirective(*MF);
More information about the llvm-commits
mailing list