aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
These changes all look good to me, but it would be best to split the commit into three parts: one for include what you use, one for modernize-use-bool-literals, and one for the copy pasta error. Keeping the changes logically separate helps in case we need to roll any particular change back for some reason (same is true for similar patches in the future; they should probably be multiple patch requests). Repository: rL LLVM https://reviews.llvm.org/D26176 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
