[PATCH] D36998: [AST] Traverse templates in LexicallyOrderedRecursiveASTVisitor

[Alex Lorenz via Phabricator via cfe-commits]

arphaman added a comment.

Awesome, thanks!



================
Comment at: unittests/Tooling/LexicallyOrderedRecursiveASTVisitorTest.cpp:146
+  DummyMatchVisitor Visitor;
+  Visitor.ExpectMatch("/f/T", 2, 11);
+  Visitor.ExpectMatch("/f/f/", 2, 20);
----------------
johannes wrote:
> This test also works before the change; what is the best way to enforce an ordering here?
You can add a flag to `LexicallyOrderedDeclVisitor` that emits the ordering index when it's true and turn it on for this test. Then you'd should get something like `/f/T at 0` and `/f/T at 1`.

Please test a class template as well.




https://reviews.llvm.org/D36998