dberris added a comment. In https://reviews.llvm.org/D45243#1056549, @devnexen wrote:
> Do the unit tests have same outcome with these changes ? Otherwise, > appreciate the simplification. Yes, locally, `ninja check-all` on Linux works just fine for me. No idea whether this will be the case for the buildbots, but that's easy enough to address when I get LGTM for the patch (and it lands, preferably daytime in Sydney). https://reviews.llvm.org/D45243 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits