[PATCH] D120949: [clang][AST matchers] adding attributedStmt AST matcher
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 30 12:56:32 PDT 2022
aaron.ballman added a comment.
In D120949#3417545 <https://reviews.llvm.org/D120949#3417545>, @ajohnson-uoregon wrote:
> Okay, I put the assert back in and made a thing similar to HasSizeMatcher that works and all the tests pass.
>
> What do you mean by a release note, in the commit message, in the docs...?
https://github.com/llvm/llvm-project/blob/main/clang/docs/ReleaseNotes.rst has a section for AST Matchers, so you should add the notes there.
Otherwise, the changes here all LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D120949/new/
https://reviews.llvm.org/D120949
More information about the cfe-commits
mailing list