[PATCH] D118900: [ASTMatchers] Expand isInline matcher to VarDecl

Evgeny Shulgin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 4 10:27:50 PST 2022


Izaron added inline comments.


================
Comment at: clang/docs/LibASTMatchersReference.html:5736
+<tr><td>Matcher<<a href="https://clang.llvm.org/doxygen/classclang_1_1VarDecl.html">VarDecl</a>></td><td class="name" onclick="toggle('isInline2')"><a name="isInline2Anchor">isInline</a></td><td></td></tr>
+<tr><td colspan="4" class="doc" id="isInline2"><pre>Matches function and namespace declarations that are marked with
+the inline keyword.
----------------
Izaron wrote:
> > Matches function and namespace declarations
> Could it be something like
> > Matches variable, function and namespace declarations
> ?
> 
> Also, maybe it worth to mention the checker in release notes? In `clang/docs/ReleaseNotes.rst`
(The matcher, not checker, of course)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118900



More information about the cfe-commits mailing list