<div dir="ltr">Test cases - and also, ideally, it'd be good to find a narrower place to do this - having to do it in 5 different places/ways seems like we might easily be missing places (or miss them in the future)?</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 6, 2016 at 12:53 PM, Nirav Dave via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">niravd created this revision.<br>
niravd added a reviewer: jyknight.<br>
niravd added a subscriber: llvm-commits.<br>
<br>
[DAG] Previously debug values would transfer debuginfo for the<br>
selected start node for a replacement which allows for dropped debug<br>
information. Push debug value transfer to value replacement.<br>
<br>
<a href="http://reviews.llvm.org/D21037" rel="noreferrer" target="_blank">http://reviews.llvm.org/D21037</a><br>
<br>
Files:<br>
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp<br>
  lib/CodeGen/SelectionDAG/SelectionDAG.cpp<br>
<br>
Index: lib/CodeGen/SelectionDAG/SelectionDAG.cpp<br>
===================================================================<br>
--- lib/CodeGen/SelectionDAG/SelectionDAG.cpp<br>
+++ lib/CodeGen/SelectionDAG/SelectionDAG.cpp<br>
@@ -6359,6 +6359,9 @@<br>
     AddModifiedNodeToCSEMaps(User);<br>
   }<br>
<br>
+  // Preserve Debug Values<br>
+  TransferDbgValues(FromN, To);<br>
+<br>
   // If we just RAUW'd the root, take note.<br>
   if (FromN == getRoot())<br>
     setRoot(To);<br>
@@ -6382,6 +6385,11 @@<br>
   if (From == To)<br>
     return;<br>
<br>
+  // Preserve Debug Info. Only do this if there's a use.<br>
+  for (unsigned i = 0, e = From->getNumValues(); i != e; ++i)<br>
+    if (From->hasAnyUseOfValue(i))<br>
+      TransferDbgValues(SDValue(From, i), SDValue(To, i));<br>
+<br>
   // Iterate over just the existing users of From. See the comments in<br>
   // the ReplaceAllUsesWith above.<br>
   SDNode::use_iterator UI = From->use_begin(), UE = From->use_end();<br>
@@ -6421,6 +6429,10 @@<br>
   if (From->getNumValues() == 1)  // Handle the simple case efficiently.<br>
     return ReplaceAllUsesWith(SDValue(From, 0), To[0]);<br>
<br>
+  // Preserve Debug Info.<br>
+  for (unsigned i = 0, e = From->getNumValues(); i != e; ++i)<br>
+    TransferDbgValues(SDValue(From, i), *To);<br>
+<br>
   // Iterate over just the existing users of From. See the comments in<br>
   // the ReplaceAllUsesWith above.<br>
   SDNode::use_iterator UI = From->use_begin(), UE = From->use_end();<br>
@@ -6508,6 +6520,9 @@<br>
     AddModifiedNodeToCSEMaps(User);<br>
   }<br>
<br>
+  // Preserve Debug Info.<br>
+  TransferDbgValues(From, To);<br>
+<br>
   // If we just RAUW'd the root, take note.<br>
   if (From == getRoot())<br>
     setRoot(To);<br>
@@ -6539,6 +6554,8 @@<br>
   if (Num == 1)<br>
     return ReplaceAllUsesOfValueWith(*From, *To);<br>
<br>
+  TransferDbgValues(*From, *To);<br>
+<br>
   // Read up all the uses and make records of them. This helps<br>
   // processing new uses that are introduced during the<br>
   // replacement process.<br>
Index: lib/CodeGen/SelectionDAG/DAGCombiner.cpp<br>
===================================================================<br>
--- lib/CodeGen/SelectionDAG/DAGCombiner.cpp<br>
+++ lib/CodeGen/SelectionDAG/DAGCombiner.cpp<br>
@@ -1331,8 +1331,6 @@<br>
     DEBUG(dbgs() << " ... into: ";<br>
           RV.getNode()->dump(&DAG));<br>
<br>
-    // Transfer debug value.<br>
-    DAG.TransferDbgValues(SDValue(N, 0), RV);<br>
     if (N->getNumValues() == RV.getNode()->getNumValues())<br>
       DAG.ReplaceAllUsesWith(N, RV.getNode());<br>
     else {<br>
<br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>