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

2022-02-23 Thread Nathan James via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc34d89818341: [ASTMatchers] Expand isInline matcher to VarDecl (authored by njames93). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

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

2022-02-22 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118900/new/ https://reviews.llvm.org/D118900

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

2022-02-20 Thread Nathan James via Phabricator via cfe-commits
njames93 updated this revision to Diff 410186. njames93 marked an inline comment as done. njames93 added a comment. Update description and release notes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118900/new/ https://reviews.llvm.org/D118900

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

2022-02-04 Thread Evgeny Shulgin via Phabricator via cfe-commits
Izaron added inline comments. Comment at: clang/docs/LibASTMatchersReference.html:5736 +Matcherhttps://clang.llvm.org/doxygen/classclang_1_1VarDecl.html;>VarDeclisInline +Matches function and namespace declarations that are marked with +the inline keyword.

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

2022-02-04 Thread Evgeny Shulgin via Phabricator via cfe-commits
Izaron added inline comments. Comment at: clang/docs/LibASTMatchersReference.html:5736 +Matcherhttps://clang.llvm.org/doxygen/classclang_1_1VarDecl.html;>VarDeclisInline +Matches function and namespace declarations that are marked with +the inline keyword. >

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

2022-02-04 Thread Evgeny Shulgin via Phabricator via cfe-commits
Izaron added a comment. Thanks for the matcher! I'm planning to reuse it in my checker (D118743 ). Looking forward to seeing this commit merged. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118900/new/

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

2022-02-03 Thread Nathan James via Phabricator via cfe-commits
njames93 created this revision. njames93 added reviewers: klimek, aaron.ballman. njames93 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Add support to the `isInline` matcher for C++17's inline variables. Repository: rG LLVM Github