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

Reply via email to