<div dir="ltr">Hi Tobias,<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="font-size:12.8px">Very nice that you cite some paper, but which exactly is the "second</span><br style="font-size:12.8px"><span style="font-size:12.8px">paper"? I could not find a reference in the code.</span><br style="font-size:12.8px"><span class="gmail-im" style="font-size:12.8px"><br>> +  // Handle deletions that make destination node unreachable.<br>> +  // (Based on the lemma 2.7 from the second paper.)<br><br></span><span style="font-size:12.8px">I also miss the second paper here</span></blockquote><div><br></div><div>There are references to two papers on the top of the file.<br>What's the best way to cite them in comments? Do you have some examples on how it's done in other places?<br><br>Thanks,<br>Kuba </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 20, 2017 at 10:38 AM, Tobias Grosser <span dir="ltr"><<a href="mailto:grosser@fastmail.com" target="_blank">grosser@fastmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jakub,<br>
<br>
some small post-commit comments:<br>
<br>
On Fri, Jul 14, 2017, at 11:58 PM, Jakub Kuderski via llvm-commits<br>
wrote:<br>
<span class="">> Author: kuhar<br>
> Date: Fri Jul 14 14:58:53 2017<br>
> New Revision: 308062<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=308062&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=308062&view=rev</a><br>
> Log:<br>
> [Dominators] Implement incremental deletions<br>
</span><span class="">> +  /// tree is informed about makes some (domominator) subtree<br>
> unreachable -- in<br>
<br>
</span>dominator<br>
<span class=""><br>
> +  // Checks if a node has proper support, as defined on the page 3 and<br>
> later<br>
> +  // explained on the page 7 of the second paper.<br>
<br>
</span>Very nice that you cite some paper, but which exactly is the "second<br>
paper"? I could not find a reference in the code.<br>
<span class=""><br>
> +  // Handle deletions that make destination node unreachable.<br>
> +  // (Based on the lemma 2.7 from the second paper.)<br>
<br>
</span>I also miss the second paper here.<br>
<span class=""><br>
> +    // Identify the top of the subtree to rebuilt by finding the NCD of<br>
<br>
</span>rebuild<br>
<br>
Best,<br>
Tobias<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div>Jakub Kuderski</div></div>
</div>