<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 12 October 2017 at 15:11, Roman Lebedev via Phabricator via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">lebedev.ri reopened this revision.<br>
lebedev.ri added a comment.<br>
<span class="">This revision is now accepted and ready to land.<br>
<br>
</span>Reverted due to <a href="http://bb9.pgr.jp/#/builders/20/builds/59" rel="noreferrer" target="_blank">http://bb9.pgr.jp/#/builders/<wbr>20/builds/59</a> that i don't currently know how to deal with.<br>
It is really sad that i failed to encounter it during testing.</blockquote><div><br></div><div>I see three issues there:</div><div><br></div><div>1) A warning in this code due to missing parentheses around a ^ operator.</div><div>2) This code generating correct warnings in the libc++ test suite. You could ask EricWF (cc'd) to look at those and either fix them or turn the warning flag off for libc++'s tests.</div><div>3) A stage2 / stage3 comparison failure in CGAtomic.cpp. That's pre-existing and nothing to do with your change. </div></div></div></div>