[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