<p dir="ltr">I'm actually curious why this function goes out of the way to detect when both things passed to it are completely irrelevant. Shouldn't the callers be doing this?</p>
<br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 11, 2016, 4:15 PM Xinliang David Li via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I was thinking that might be reason -- that is why I need a fresh eye<br>
to look at it :)<br>
<br>
thanks,<br>
<br>
David<br>
<br>
<br>
On Thu, Aug 11, 2016 at 3:52 PM, Michael Kuperstein <<a href="mailto:mkuper@google.com" target="_blank">mkuper@google.com</a>> wrote:<br>
> mkuper requested changes to this revision.<br>
> mkuper added a comment.<br>
> This revision now requires changes to proceed.<br>
><br>
> It looks like this is on purpose:<br>
><br>
> <a href="https://reviews.llvm.org/rL68047" rel="noreferrer" target="_blank">https://reviews.llvm.org/rL68047</a><br>
> "getEntryFor() may invalidate DenseMap iterator.<br>
> Walking an invalidated iterator is not a good idea."<br>
><br>
> I think this is still true, but we should probably add a comment.<br>
><br>
><br>
> <a href="https://reviews.llvm.org/D23431" rel="noreferrer" target="_blank">https://reviews.llvm.org/D23431</a><br>
><br>
><br>
><br>
_______________________________________________<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/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>