gribozavr2 added a comment.

Please run clang/docs/tools/dump_ast_matchers.py to update the docs.



================
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:7022
+      .matches(Node, Finder, Builder);
+}
+
----------------
Could you move it closer to other parameter-related matchers, maybe near the 
definition of `forEachArgumentWithParam`?


================
Comment at: clang/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp:2650
+  EXPECT_TRUE(matches("void x(int a, int b) {}", 
parmVarDecl(isAtPosition(1))));
+  EXPECT_TRUE(notMatches("void x(int val) {}", parmVarDecl(isAtPosition(1))));
+}
----------------
Could you change some tests to have a function declaration instead of 
definition?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80603/new/

https://reviews.llvm.org/D80603



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to