<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 13, 2014 at 2:57 PM, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">> .. of the patch. The paragraph is confusing, as I don't see why Clang would<br>
> need to change with this patch as it doesn't set weak to true...<br>
<br>
</div>It has to adapt to the new return type of cmpxchg instructions.<br>
Changing all the current icmps to extractvalues and adding a bunch<br>
more for when the loaded value is used.<br></blockquote><div><br></div><div>Ah, yea, so its not about weak, its about result extraction. Makes sense.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
I'm attaching it here in case you're interested, though it's all<br>
pretty straightforward.<br></blockquote><div><br></div><div>LGTM, but please submit the CHECK-LABEL-ification as its own patch first? =]</div></div></div></div>