[PATCH] D140794: [ASTMatcher] Add coroutineBodyStmt matcher
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 27 16:26:58 PST 2023
sammccall added a comment.
It seems pretty weird to me that the two child edges from functiondecl -> coroutinebodystmt -> compoundstmt are both called "body".
However that *is* what they're called in the AST today, and having the matchers correspond to the AST's names seems important.
So this seems OK to me (renaming the inner "body" edge to something else seems like a good idea, but is out of scope here).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140794/new/
https://reviews.llvm.org/D140794
More information about the cfe-commits
mailing list