On 12 October 2017 at 15:11, Roman Lebedev via Phabricator via cfe-commits < cfe-commits@lists.llvm.org> wrote:
> lebedev.ri reopened this revision. > lebedev.ri added a comment. > This revision is now accepted and ready to land. > > Reverted due to http://bb9.pgr.jp/#/builders/20/builds/59 that i don't > currently know how to deal with. > It is really sad that i failed to encounter it during testing. I see three issues there: 1) A warning in this code due to missing parentheses around a ^ operator. 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. 3) A stage2 / stage3 comparison failure in CGAtomic.cpp. That's pre-existing and nothing to do with your change.
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
