<p dir="ltr">Imho:  It should be trivial to fix this, and we shouldn't hack around it. </p>
<br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 6, 2017, 10:46 PM Davide Italiano 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">davide added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D38586#891283" rel="noreferrer" target="_blank">https://reviews.llvm.org/D38586#891283</a>, @chapuni wrote:<br>
<br>
> The test added. This comes from EmitAtomicUpdateValue() in CGAtomic.cpp.<br>
>  I confirmed it fails on both libstdc++ and libc++ to me.<br>
><br>
> I will attempt to get rid of O(N^2) there, but I think it'd not be big deal unless preds exceeded hundreds.<br>
<br>
<br>
I really think we should try to at least make an attempt to get rid of the O(N^2) behaviour as it doesn't seem terribly complicated to do so _and_ it may bite us in the future in many unexpected ways. The test seems fine, but you can probably clean it up a bit (and run metarenamer on it for shorter names)<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D38586" rel="noreferrer" target="_blank">https://reviews.llvm.org/D38586</a><br>
<br>
<br>
<br>
</blockquote></div>