[PATCH] D44356: [LICM] Salvage DebugInfo from dying Instructions

[Anastasis via Phabricator via llvm-commits]

gramanas updated this revision to Diff 138217.
gramanas marked 2 inline comments as done.
gramanas added a comment.

- Fix test label


Repository:
  rL LLVM

https://reviews.llvm.org/D44356

Files:
  lib/Transforms/Scalar/LICM.cpp
  test/Transforms/LICM/sinking.ll


Index: test/Transforms/LICM/sinking.ll
===================================================================
--- test/Transforms/LICM/sinking.ll
+++ test/Transforms/LICM/sinking.ll
@@ -1,4 +1,5 @@
 ; RUN: opt < %s -basicaa -licm -S | FileCheck %s
+; RUN: opt < %s -debugify -basicaa -licm -S | FileCheck %s -check-prefix=DEBUGIFY
 
 declare i32 @strlen(i8*) readonly nounwind
 
@@ -249,6 +250,9 @@
 ; CHECK-LABEL: @test11(
 ; CHECK:     Out:
 ; CHECK-NEXT:  ret void
+
+; DEBUGIFY-LABEL: @test11(
+; DEBUGIFY: call void @llvm.dbg.value(metadata %Ty* @X2, metadata {{.*}}, metadata !DIExpression(DW_OP_stack_value))
 }
 
 @c = common global [1 x i32] zeroinitializer, align 4
Index: lib/Transforms/Scalar/LICM.cpp
===================================================================
--- lib/Transforms/Scalar/LICM.cpp
+++ lib/Transforms/Scalar/LICM.cpp
@@ -393,6 +393,7 @@
       // used in the loop, instead, just delete it.
       if (isInstructionTriviallyDead(&I, TLI)) {
         DEBUG(dbgs() << "LICM deleting dead inst: " << I << '\n');
+        salvageDebugInfo(I);
         ++II;
         CurAST->deleteValue(&I);
         I.eraseFromParent();


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44356.138217.patch
Type: text/x-patch
Size: 1149 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180313/237bdf64/attachment.bin>