mikecrowe added inline comments.
================
Comment at: clang/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp:1671
+ notMatches("void x(int, int = 1) { x(0); }",
+ traverse(TK_IgnoreUnlessSpelledInSource, Call2PlusArgs)));
+ EXPECT_TRUE(
----------------
I'm not sure if this is the best way to trigger testing of the ignoring default
argument path in the implementation of `argumentCountAtLeast`, but it does seem
to work. I added explicit `TK_AsIs` to the tests above to make it clear that
they are not testing that path.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153716/new/
https://reviews.llvm.org/D153716
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
