The inliner needs to fix up debug information for llvm.dbg.declare, not only for...

Patch by Amjad Aboud

Differential Revision: http://reviews.llvm.org/D6525

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@224015 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Transforms/Utils/InlineFunction.cpp b/lib/Transforms/Utils/InlineFunction.cpp
index 147237edbcb586ba224d83c4650627c95eafcde7..0441d9721cc421cb4245bd82f3e847669324fbbc 100644 (file)
--- a/lib/Transforms/Utils/InlineFunction.cpp
+++ b/lib/Transforms/Utils/InlineFunction.cpp
@@ -872,6 +872,12 @@ static void fixupLineNumbers(Function *Fn, Function::iterator FI,
           DVI->setOperand(2, MetadataAsValue::get(
                                  Ctx, createInlinedVariable(DVI->getVariable(),
                                                             InlinedAt, Ctx)));
+        } else if (DbgDeclareInst *DDI = dyn_cast<DbgDeclareInst>(BI)) {
+          LLVMContext &Ctx = BI->getContext();
+          MDNode *InlinedAt = BI->getDebugLoc().getInlinedAt(Ctx);
+          DDI->setOperand(1, MetadataAsValue::get(
+                                 Ctx, createInlinedVariable(DDI->getVariable(),
+                                                            InlinedAt, Ctx)));
         }
       }
     }

diff --git a/test/Transforms/Inline/inline_dbg_declare.ll b/test/Transforms/Inline/inline_dbg_declare.ll
new file mode 100644 (file)
index 0000000..52d3dbf
--- /dev/null
+++ b/test/Transforms/Inline/inline_dbg_declare.ll
@@ -0,0 +1,96 @@
+; RUN: opt < %s -S -inline | FileCheck %s\r
+;\r
+; The purpose of this test is to check that inline pass preserves debug info\r
+; for variable using the dbg.declare intrinsic.\r
+;\r
+;; This test was generated by running this command:\r
+;; clang.exe -S -O0 -emit-llvm -g foo.c\r
+;;\r
+;; foo.c\r
+;; ==========================\r
+;; float foo(float x)\r
+;; {\r
+;;    return x;\r
+;; }\r
+;;\r
+;; void bar(float *dst)\r
+;; {\r
+;;    dst[0] = foo(dst[0]);\r
+;; }\r
+;; ==========================\r
+\r
+target datalayout = "e-m:w-p:32:32-i64:64-f80:32-n8:16:32-S32"\r
+target triple = "i686-pc-windows-msvc"\r
+\r
+; Function Attrs: nounwind\r
+define float @foo(float %x) #0 {\r
+entry:\r
+  %x.addr = alloca float, align 4\r
+  store float %x, float* %x.addr, align 4\r
+  call void @llvm.dbg.declare(metadata !{float* %x.addr}, metadata !16, metadata !17), !dbg !18\r
+  %0 = load float* %x.addr, align 4, !dbg !19\r
+  ret float %0, !dbg !19\r
+}\r
+\r
+; Function Attrs: nounwind readnone\r
+declare void @llvm.dbg.declare(metadata, metadata, metadata) #1\r
+\r
+; CHECK: define void @bar\r
+\r
+; Function Attrs: nounwind\r
+define void @bar(float* %dst) #0 {\r
+entry:\r
+\r
+; CHECK: [[x_addr_i:%[a-zA-Z0-9.]+]] = alloca float, align 4\r
+\r
+  %dst.addr = alloca float*, align 4\r
+  store float* %dst, float** %dst.addr, align 4\r
+  call void @llvm.dbg.declare(metadata !{float** %dst.addr}, metadata !20, metadata !17), !dbg !21\r
+  %0 = load float** %dst.addr, align 4, !dbg !22\r
+  %arrayidx = getelementptr inbounds float* %0, i32 0, !dbg !22\r
+  %1 = load float* %arrayidx, align 4, !dbg !22\r
+  %call = call float @foo(float %1), !dbg !22\r
+\r
+; CHECK-NOT: call float @foo\r
+; CHECK: void @llvm.dbg.declare(metadata !{float* [[x_addr_i]]}, metadata [[m23:![0-9]+]], metadata !17), !dbg [[m24:![0-9]+]]\r
+\r
+  %2 = load float** %dst.addr, align 4, !dbg !22\r
+  %arrayidx1 = getelementptr inbounds float* %2, i32 0, !dbg !22\r
+  store float %call, float* %arrayidx1, align 4, !dbg !22\r
+  ret void, !dbg !23\r
+}\r
+\r
+attributes #0 = { nounwind }\r
+attributes #1 = { nounwind readnone }\r
+\r
+!llvm.dbg.cu = !{!0}\r
+!llvm.module.flags = !{!13, !14}\r
+!llvm.ident = !{!15}\r
+\r
+!0 = metadata !{metadata !"0x11\0012\00clang version 3.6.0 (trunk)\000\00\000\00\001", metadata !1, metadata !2, metadata !2, metadata !3, metadata !2, metadata !2} ; [ DW_TAG_compile_unit ] [foo.c] [DW_LANG_C99]\r
+!1 = metadata !{metadata !"foo.c", metadata !""}\r
+!2 = metadata !{}\r
+!3 = metadata !{metadata !4, metadata !9}\r
+!4 = metadata !{metadata !"0x2e\00foo\00foo\00\001\000\001\000\000\00256\000\002", metadata !1, metadata !5, metadata !6, null, float (float)* @foo, null, null, metadata !2} ; [ DW_TAG_subprogram ] [line 1] [def] [scope 2] [foo]\r
+!5 = metadata !{metadata !"0x29", metadata !1}    ; [ DW_TAG_file_type ] [foo.c]\r
+!6 = metadata !{metadata !"0x15\00\000\000\000\000\000\000", null, null, null, metadata !7, null, null, null} ; [ DW_TAG_subroutine_type ] [line 0, size 0, align 0, offset 0] [from ]\r
+!7 = metadata !{metadata !8, metadata !8}\r
+!8 = metadata !{metadata !"0x24\00float\000\0032\0032\000\000\004", null, null} ; [ DW_TAG_base_type ] [float] [line 0, size 32, align 32, offset 0, enc DW_ATE_float]\r
+!9 = metadata !{metadata !"0x2e\00bar\00bar\00\006\000\001\000\000\00256\000\007", metadata !1, metadata !5, metadata !10, null, void (float*)* @bar, null, null, metadata !2} ; [ DW_TAG_subprogram ] [line 6] [def] [scope 7] [bar]\r
+!10 = metadata !{metadata !"0x15\00\000\000\000\000\000\000", null, null, null, metadata !11, null, null, null} ; [ DW_TAG_subroutine_type ] [line 0, size 0, align 0, offset 0] [from ]\r
+!11 = metadata !{null, metadata !12}\r
+!12 = metadata !{metadata !"0xf\00\000\0032\0032\000\000", null, null, metadata !8} ; [ DW_TAG_pointer_type ] [line 0, size 32, align 32, offset 0] [from float]\r
+!13 = metadata !{i32 2, metadata !"Dwarf Version", i32 4}\r
+!14 = metadata !{i32 2, metadata !"Debug Info Version", i32 2}\r
+!15 = metadata !{metadata !"clang version 3.6.0 (trunk)"}\r
+!16 = metadata !{metadata !"0x101\00x\0016777217\000", metadata !4, metadata !5, metadata !8} ; [ DW_TAG_arg_variable ] [x] [line 1]\r
+!17 = metadata !{metadata !"0x102"}               ; [ DW_TAG_expression ]\r
+!18 = metadata !{i32 1, i32 17, metadata !4, null}\r
+!19 = metadata !{i32 3, i32 5, metadata !4, null}\r
+!20 = metadata !{metadata !"0x101\00dst\0016777222\000", metadata !9, metadata !5, metadata !12} ; [ DW_TAG_arg_variable ] [dst] [line 6]\r
+!21 = metadata !{i32 6, i32 17, metadata !9, null}\r
+!22 = metadata !{i32 8, i32 14, metadata !9, null}\r
+!23 = metadata !{i32 9, i32 1, metadata !9, null}\r
+\r
+; CHECK: [[m23]] = metadata !{metadata !"0x101\00x\0016777217\000", metadata !4, metadata !5, metadata !8, metadata !22} ; [ DW_TAG_arg_variable ] [x] [line 1]\r
+; CHECK: [[m24]] = metadata !{i32 1, i32 17, metadata !4, metadata !22}\r