[llvm] r283500 - Preserve the debug location when CodeGenPrepare sinks a compare instruction into the

[Wolfgang Pieb via llvm-commits]

Author: wolfgangp
Date: Thu Oct  6 16:43:45 2016
New Revision: 283500

URL: http://llvm.org/viewvc/llvm-project?rev=283500&view=rev
Log:
Preserve the debug location when CodeGenPrepare sinks a compare instruction into the
basic block of a user.

Patch by Andrea DiBiagio.

Differential Revision: https://reviews.llvm.org/D24632


Added:
    llvm/trunk/test/DebugInfo/Generic/sunk-compare.ll
Modified:
    llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp

Modified: llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp?rev=283500&r1=283499&r2=283500&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp (original)
+++ llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp Thu Oct  6 16:43:45 2016
@@ -926,6 +926,8 @@ static bool SinkCmpExpression(CmpInst *C
       InsertedCmp =
           CmpInst::Create(CI->getOpcode(), CI->getPredicate(),
                           CI->getOperand(0), CI->getOperand(1), "", &*InsertPt);
+      // Propagate the debug info.
+      InsertedCmp->setDebugLoc(CI->getDebugLoc());
     }
 
     // Replace a use of the cmp with a use of the new cmp.

Added: llvm/trunk/test/DebugInfo/Generic/sunk-compare.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/Generic/sunk-compare.ll?rev=283500&view=auto
==============================================================================
--- llvm/trunk/test/DebugInfo/Generic/sunk-compare.ll (added)
+++ llvm/trunk/test/DebugInfo/Generic/sunk-compare.ll Thu Oct  6 16:43:45 2016
@@ -0,0 +1,46 @@
+; RUN: opt -S -codegenprepare < %s | FileCheck %s
+;
+; This test case has been generated by hand but is inspired by the
+; observation that compares that are sunk into the basic blocks where
+; their results are used did not retain their debug locs. This caused
+; sample profiling to attribute code to the wrong source lines. 
+;
+; We check that the compare instruction retains its debug loc after 
+; it is sunk into other.bb by the codegen prepare pass.
+; 
+; CHECK:       other.bb:
+; CHECK-NEXT:  icmp{{.*}}%x, 0, !dbg ![[MDHANDLE:[0-9]*]]
+; CHECK:       ![[MDHANDLE]] = !DILocation(line: 2
+;
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
+
+define i32 @_Z3fooii(i32 %x, i32 %y) !dbg !5 {
+entry:
+  %cmp17 = icmp sgt i32 %x, 0, !dbg !6
+  br label %other.bb, !dbg !6
+
+other.bb:
+  br i1 %cmp17, label %exit1.bb, label %exit2.bb, !dbg !7
+
+exit1.bb:
+  %0 = add i32 %y, 42, !dbg !8
+  ret i32 %0, !dbg !8
+
+exit2.bb:
+  ret i32 44, !dbg !9
+
+}
+
+!llvm.dbg.cu = !{!0}
+!llvm.module.flags = !{!3, !4}
+
+!0 = distinct !DICompileUnit(language: DW_LANG_C, file: !1, producer: "clang", isOptimized: true, runtimeVersion: 0, emissionKind: FullDebug, enums: !2, globals: !2)
+!1 = !DIFile(filename: "test.cpp", directory: "/debuginfo/bug/cgp")
+!2 = !{}
+!3 = !{i32 2, !"Dwarf Version", i32 4}
+!4 = !{i32 2, !"Debug Info Version", i32 3}
+!5 = distinct !DISubprogram(name: "foo", linkageName: "foo", scope: !1, file: !1, line: 1, isLocal: false, isDefinition: true, scopeLine: 1, flags: DIFlagPrototyped, isOptimized: false, unit: !0, variables: !2)
+!6 = !DILocation(line: 2, column: 0, scope: !5)
+!7 = !DILocation(line: 3, column: 0, scope: !5)
+!8 = !DILocation(line: 4, column: 0, scope: !5)
+!9 = !DILocation(line: 5, column: 0, scope: !5)