[PATCH] D49701: [ASTMatchers] Introduce a matcher for `ObjCIvarExpr`, support getting it's declaration

[Aaron Ballman via Phabricator via cfe-commits]

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