[clang] [ASTMatchers][Docs] print ignoring message only when class was not documented before (PR #203783)

[via cfe-commits]

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