<div dir="ltr">I fixed the typos in r309170<span style="font-size:17.6px">.</span><br><br>Best,<br>Kuba</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 21, 2017 at 3:45 AM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias.grosser@inf.ethz.ch" target="_blank">tobias.grosser@inf.ethz.ch</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Jul 20, 2017, at 07:38 PM, Tobias Grosser via llvm-commits<br>
wrote:<br>
<span class="">> 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>
> > 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>
> > +  /// tree is informed about makes some (domominator) subtree<br>
> > unreachable -- in<br>
><br>
> dominator<br>
><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>
> 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>
<br>
</span>Sorry, it seems I did not pay enough attention. I now get it.<br>
<br>
I would probably write "see paper [2] at the top of the file" and index<br>
the papers accordingly. However, this is really now only a minor<br>
difference.<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>