<div dir="auto">Wait. Scratch that. This is not GatherAllAliases. I keep making that mistake. <div dir="auto"><br></div><div dir="auto">That said there is a potential regression from lifetime nodes because they can form an artificial dependence between ops before and after it. </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 26, 2019, 16:28 Nirav Dave via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">niravd added inline comments.<br>
<br>
<br>
================<br>
Comment at: llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:15524<br>
+        Chains.push_back(Chain.getOperand(0));<br>
+      }<br>
+      break;<br>
----------------<br>
Ahha! We never add the lifetime to the Alias list like we should if we dont' skip past it. I suspect this is why we had to revert this patch.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D57541/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D57541/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D57541" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D57541</a><br>
<br>
<br>
<br>
</blockquote></div>