llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Baranov Victor (vbvictor) <details> <summary>Changes</summary> --- Full diff: https://github.com/llvm/llvm-project/pull/203783.diff 1 Files Affected: - (modified) clang/docs/tools/dump_ast_matchers.py (+4-1) ``````````diff diff --git a/clang/docs/tools/dump_ast_matchers.py b/clang/docs/tools/dump_ast_matchers.py index 5db6826070934..5bd7b728a1c56 100755 --- a/clang/docs/tools/dump_ast_matchers.py +++ b/clang/docs/tools/dump_ast_matchers.py @@ -521,7 +521,10 @@ def act_on_decl(declaration, comment, allowed_types): if not result_types: if not comment: # Only overloads don't have their own doxygen comments; ignore those. - print('Ignoring "%s"' % name) + # Warn if this name was never successfully documented. + # Overloads of an already-documented matcher are expected. + if ids[name] == 0: + print('Ignoring "%s"' % name) else: print('Cannot determine result type for "%s"' % name) else: `````````` </details> https://github.com/llvm/llvm-project/pull/203783 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
