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@0` and `/f/T@1`.
Please test a class template as well.
https://reviews.llvm.org/D36998
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
