[PATCH] D46348: [SelectionDAG] Transfer DbgValues when casts are optimized in SelectionDAG::getNode

[Aaron Smith via Phabricator via llvm-commits]

asmith created this revision.
asmith added a reviewer: aprantl.
Herald added a reviewer: javed.absar.
Herald added subscribers: llvm-commits, JDevlieghere, kristof.beyls.

getNode optimizes (ext (trunc x)) to x and the dbgvalue node on trunc is lost. The fix calls transferDbgValues to add the dbgvalue to x.

Add DebugInfo/AArch64/dbg-value-i16.ll

Patch by Sejong Oh!


Repository:
  rL LLVM

https://reviews.llvm.org/D46348

Files:
  lib/CodeGen/SelectionDAG/SelectionDAG.cpp
  test/DebugInfo/AArch64/dbg-value-i16.ll


Index: test/DebugInfo/AArch64/dbg-value-i16.ll
===================================================================
--- /dev/null
+++ test/DebugInfo/AArch64/dbg-value-i16.ll
@@ -0,0 +1,51 @@
+; RUN: llc -stop-after=livedebugvalues < %s | FileCheck %s
+;
+; ModuleID = 'dbg-value-i16.ll'
+target datalayout = "e-m:e-i8:8:32-i16:16:32-i64:64-i128:128-n32:64-S128"
+target triple = "aarch64"
+
+; CHECK: DBG_VALUE
+; Function Attrs: nounwind 
+define void @f() #0 !dbg !4 {
+entry:
+  tail call void @h(i16 0) #2, !dbg !14
+  %call = tail call i16 (...) @g() #2, !dbg !15
+  tail call void @llvm.dbg.value(metadata i16 %call, metadata !8, metadata !16), !dbg !17
+  tail call void @h(i16 %call) #2, !dbg !18
+  ret void, !dbg !19
+}
+
+declare void @h(i16)
+
+declare i16 @g(...)
+
+; Function Attrs: nounwind readnone
+declare void @llvm.dbg.value(metadata, metadata, metadata) #1
+
+attributes #0 = { nounwind }
+attributes #1 = { nounwind readnone }
+attributes #2 = { nounwind }
+
+!llvm.dbg.cu = !{!0}
+!llvm.module.flags = !{!10, !11, !12}
+!llvm.ident = !{!13}
+
+!0 = distinct !DICompileUnit(language: DW_LANG_C99, file: !1, producer: "clang version 3.9.0 ", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !2)
+!1 = !DIFile(filename: "test.c", directory: "/Volumes/Data/llvm")
+!2 = !{}
+!4 = distinct !DISubprogram(name: "f", scope: !1, file: !1, line: 3, type: !5, isLocal: false, isDefinition: true, scopeLine: 3, isOptimized: true, unit: !0, variables: !7)
+!5 = !DISubroutineType(types: !6)
+!6 = !{null}
+!7 = !{!8}
+!8 = !DILocalVariable(name: "a", scope: !4, file: !1, line: 5, type: !9)
+!9 = !DIBasicType(name: "int", size: 32, align: 32, encoding: DW_ATE_signed)
+!10 = !{i32 2, !"Dwarf Version", i32 2}
+!11 = !{i32 2, !"Debug Info Version", i32 3}
+!12 = !{i32 1, !"PIC Level", i32 2}
+!13 = !{!"clang version 3.9.0 "}
+!14 = !DILocation(line: 4, column: 3, scope: !4)
+!15 = !DILocation(line: 5, column: 11, scope: !4)
+!16 = !DIExpression()
+!17 = !DILocation(line: 5, column: 7, scope: !4)
+!18 = !DILocation(line: 6, column: 3, scope: !4)
+!19 = !DILocation(line: 7, column: 1, scope: !4)
Index: lib/CodeGen/SelectionDAG/SelectionDAG.cpp
===================================================================
--- lib/CodeGen/SelectionDAG/SelectionDAG.cpp
+++ lib/CodeGen/SelectionDAG/SelectionDAG.cpp
@@ -3965,8 +3965,10 @@
     // (ext (trunx x)) -> x
     if (OpOpcode == ISD::TRUNCATE) {
       SDValue OpOp = Operand.getOperand(0);
-      if (OpOp.getValueType() == VT)
+      if (OpOp.getValueType() == VT) {
+        transferDbgValues(Operand, OpOp, 0, 0, false);
         return OpOp;
+      }
     }
     break;
   case ISD::TRUNCATE:


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46348.144830.patch
Type: text/x-patch
Size: 2684 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180502/423525cd/attachment.bin>