<div dir="ltr">Thanks. Can you also fix that above <span style="color:rgb(36,41,46);white-space:pre-wrap"><span style="font-family:SFMono-Regular,Consolas,"Liberation Mono",Menlo,Courier,monospace;font-size:12px">DeletingEdgesIntroducesI<wbr>nfiniteLoop2?

</span><font face="arial, helvetica, sans-serif">Best,
Kuba</font></span><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 2, 2017 at 9:19 PM, 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"><span>On Thu, Aug 3, 2017, at 06:12, Tobias Grosser via llvm-commits wrote:<br>
> On Thu, Aug 3, 2017, at 02:19, Jakub Kuderski via Phabricator via<br>
> llvm-commits wrote:<br>
> > kuhar added a comment.<br>
> ><br>
> > I've just realized that I didn't notice the TODOs  in this patch when I<br>
> > did a code review for it.<br>
> ><br>
> >   // TODO: Can we change the PDT definition such that C remains part of<br>
> >   the<br>
> >   //       CFG, at best without loosing the dominance relation D postdom<br>
> >   B.<br>
> ><br>
> > I don't think we reached an agreement on the second part of the TODO, as<br>
> > the `"at best"` might imply. Could you reword the second part of the<br>
> > comment to be less ambiguous and not to suggest that?<br>
><br>
> I can just drop it.There is no need to sneak in TODOs to reach an<br>
> agreement.<br>
> Hopefully our discussion is fruit<br>
><br>
> > And a very minor nitpick: isn't it supposed to be s/loosing/losing? :)<br>
<br>
</span>r309919<br>
<br>
Best,<br>
Tobias<br>
><br>
> Best,<br>
> Tobias<br>
<div class="m_-8827577420404674376HOEnZb"><div class="m_-8827577420404674376h5">> ______________________________<wbr>_________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">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/<wbr>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="m_-8827577420404674376gmail_signature" data-smartmail="gmail_signature"><div>Jakub Kuderski</div></div>
</div></div>