<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 4, 2014 at 2:29 PM, Adrian Prantl <span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: adrian<br>
Date: Thu Dec  4 16:29:04 2014<br>
New Revision: 223401<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=223401&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=223401&view=rev</a><br>
Log:<br>
Debug info: If the RegisterCoalescer::reMaterializeTrivialDef() is<br>
eliminating all uses of a vreg, update any DBG_VALUE describing that vreg<br>
to point to the rematerialized register instead.<br>
<br>
Added:<br>
    llvm/trunk/test/DebugInfo/AArch64/coalescing.ll<br>
Modified:<br>
    llvm/trunk/lib/CodeGen/RegisterCoalescer.cpp<br>
<br>
Modified: llvm/trunk/lib/CodeGen/RegisterCoalescer.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/RegisterCoalescer.cpp?rev=223401&r1=223400&r2=223401&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/RegisterCoalescer.cpp?rev=223401&r1=223400&r2=223401&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/CodeGen/RegisterCoalescer.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/RegisterCoalescer.cpp Thu Dec  4 16:29:04 2014<br>
@@ -898,8 +898,20 @@ bool RegisterCoalescer::reMaterializeTri<br>
<br>
   // The source interval can become smaller because we removed a use.<br>
   LIS->shrinkToUses(&SrcInt, &DeadDefs);<br>
-  if (!DeadDefs.empty())<br>
+  if (!DeadDefs.empty()) {<br>
+    // If the virtual SrcReg is completely eliminated, update all DBG_VALUEs<br>
+    // to describe DstReg instead.<br>
+    for (MachineRegisterInfo::use_iterator UI = MRI->use_begin(SrcReg),<br>
+         UE = MRI->use_end(); UI != UE; ++UI) {<br></blockquote><div><br>range-for with use_operands?<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+      MachineOperand &UseMO = *UI;<br>
+      MachineInstr *UseMI = UseMO.getParent();<br>
+      if (UseMI->isDebugValue()) {<br>
+        UseMO.setReg(DstReg);<br>
+        DEBUG({dbgs() << "\t\tupdated: " <<  *UseMI;});<br>
+      }<br>
+    }<br>
     eliminateDeadDefs();<br>
+  }<br>
<br>
   return true;<br>
 }<br>
<br>
Added: llvm/trunk/test/DebugInfo/AArch64/coalescing.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/AArch64/coalescing.ll?rev=223401&view=auto" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/AArch64/coalescing.ll?rev=223401&view=auto</a><br>
==============================================================================<br>
--- llvm/trunk/test/DebugInfo/AArch64/coalescing.ll (added)<br>
+++ llvm/trunk/test/DebugInfo/AArch64/coalescing.ll Thu Dec  4 16:29:04 2014<br>
@@ -0,0 +1,70 @@<br>
+; RUN: llc -filetype=obj %s -o - | llvm-dwarfdump - | FileCheck %s<br>
+;<br>
+; Generated at -Os from:<br>
+; void *my_memcpy(void *dst, const void * src, long n);<br>
+; void* getBytesNoCopy();<br>
+; void start() {<br>
+;   unsigned size;<br>
+;   my_memcpy(&size, getBytesNoCopy(), sizeof(size));<br></blockquote><div><br>are all of these parameters needed? Any idea what the interactions are?<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+;   if (size != 0) { }<br></blockquote><div><br>Sort of surprised this code isn't DCE'd by -Os?<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+; }<br>
+<br>
+; ModuleID = 'test1.cpp'<br>
+target datalayout = "e-m:o-i64:64-i128:128-n32:64-S128"<br>
+target triple = "arm64-apple-ios"<br>
+<br>
+; Function Attrs: nounwind optsize<br>
+define void @_Z5startv() #0 {<br>
+entry:<br>
+  %size = alloca i32, align 4<br>
+  %0 = bitcast i32* %size to i8*, !dbg !15<br>
+  %call = tail call i8* @_Z14getBytesNoCopyv() #3, !dbg !16<br>
+  %call1 = call i8* @_Z9my_memcpyPvPKvl(i8* %0, i8* %call, i64 4) #3, !dbg !15<br>
+  call void @llvm.dbg.value(metadata !{i32* %size}, i64 0, metadata !10, metadata !17), !dbg !18<br>
+  ; CHECK: .debug_info contents:<br>
+  ; CHECK: DW_TAG_variable<br>
+  ; CHECK-NEXT: DW_AT_location<br>
+  ; CHECK-NEXT: DW_AT_name {{.*}}"size"<br>
+  ; CHECK: .debug_loc contents:<br>
+  ; CHECK: Location description: 70 00<br>
+  ret void, !dbg !19<br>
+}<br>
+<br>
+; Function Attrs: optsize<br>
+declare i8* @_Z9my_memcpyPvPKvl(i8*, i8*, i64) #1<br>
+<br>
+; Function Attrs: optsize<br>
+declare i8* @_Z14getBytesNoCopyv() #1<br>
+<br>
+; Function Attrs: nounwind readnone<br>
+declare void @llvm.dbg.value(metadata, i64, metadata, metadata) #2<br>
+<br>
+attributes #0 = { nounwind optsize }<br>
+attributes #1 = { optsize }<br>
+attributes #2 = { nounwind readnone }<br>
+attributes #3 = { nounwind optsize }<br>
+<br>
+!<a href="http://llvm.dbg.cu" target="_blank">llvm.dbg.cu</a> = !{!0}<br>
+!llvm.module.flags = !{!12, !13}<br>
+!llvm.ident = !{!14}<br>
+<br>
+!0 = metadata !{metadata !"0x11\004\00clang version 3.6.0 (trunk 223149) (llvm/trunk 223153)\001\00\000\00\001", metadata !1, metadata !2, metadata !2, metadata !3, metadata !2, metadata !2} ; [ DW_TAG_compile_unit ] [/<stdin>] [DW_LANG_C_plus_plus]<br>
+!1 = metadata !{metadata !"<stdin>", metadata !""}<br>
+!2 = metadata !{}<br>
+!3 = metadata !{metadata !4}<br>
+!4 = metadata !{metadata !"0x2e\00start\00start\00_Z5startv\003\000\001\000\000\00256\001\004", metadata !5, metadata !6, metadata !7, null, void ()* @_Z5startv, null, null, metadata !9} ; [ DW_TAG_subprogram ] [line 3] [def] [scope 4] [start]<br>
+!5 = metadata !{metadata !"test1.cpp", metadata !""}<br>
+!6 = metadata !{metadata !"0x29", metadata !5}    ; [ DW_TAG_file_type ] [/test1.cpp]<br>
+!7 = metadata !{metadata !"0x15\00\000\000\000\000\000\000", null, null, null, metadata !8, null, null, null} ; [ DW_TAG_subroutine_type ] [line 0, size 0, align 0, offset 0] [from ]<br>
+!8 = metadata !{null}<br>
+!9 = metadata !{metadata !10}<br>
+!10 = metadata !{metadata !"0x100\00size\005\000", metadata !4, metadata !6, metadata !11} ; [ DW_TAG_auto_variable ] [size] [line 5]<br>
+!11 = metadata !{metadata !"0x24\00unsigned int\000\0032\0032\000\000\007", null, null} ; [ DW_TAG_base_type ] [unsigned int] [line 0, size 32, align 32, offset 0, enc DW_ATE_unsigned]<br>
+!12 = metadata !{i32 2, metadata !"Dwarf Version", i32 2}<br>
+!13 = metadata !{i32 2, metadata !"Debug Info Version", i32 2}<br>
+!14 = metadata !{metadata !"clang version 3.6.0 (trunk 223149) (llvm/trunk 223153)"}<br>
+!15 = metadata !{i32 6, i32 3, metadata !4, null}<br>
+!16 = metadata !{i32 6, i32 25, metadata !4, null}<br>
+!17 = metadata !{metadata !"0x102"}               ; [ DW_TAG_expression ]<br>
+!18 = metadata !{i32 5, i32 12, metadata !4, null}<br>
+!19 = metadata !{i32 9, i32 1, metadata !4, null}<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div></div>