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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to