aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM! ================ Comment at: clang/include/clang/ASTMatchers/ASTMatchersInternal.h:44 #include "clang/AST/Expr.h" +#include "clang/AST/ExprObjC.h" #include "clang/AST/ExprCXX.h" ---------------- george.karpenkov wrote: > aaron.ballman wrote: > > This list should remain sorted alphabetically. > but it is? That's really odd. For me, the patch view shows: ``` ================ Comment at: clang/include/clang/ASTMatchers/ASTMatchersInternal.h:44 #include "clang/AST/Expr.h" +#include "clang/AST/ExprObjC.h" #include "clang/AST/ExprCXX.h" ---------------- aaron.ballman wrote: > This list should remain sorted alphabetically. but it is? ``` but the Phabricator view shows it in the correct order. https://reviews.llvm.org/D49701 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
