This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG9809c6c61ceb: Add `isInitCapture` and `forEachLambdaCapture`
matchers. (authored by jcking1034, committed by ymandel).
Repository:
rG LLVM Github
aaron.ballman accepted this revision.
aaron.ballman added a comment.
LGTM!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113575/new/
https://reviews.llvm.org/D113575
___
cfe-commits mailing list
cfe-comm
ymandel accepted this revision.
ymandel added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:4235
+ for (const auto &Capture : Node.captures()) {
+if (Finder->isTraversalIgnoringImplicitNodes() && C
fowles added a comment.
That is awesome! Thanks for much for the examples. Everything here LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113575/new/
https://reviews.llvm.org/D113575
___
cfe-commi
jcking1034 marked an inline comment as done.
jcking1034 added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:4224
+/// float z;
+/// auto f = [=]() { return x + y + z; };
+/// }
fowles wrote:
> it would be nice to be able to
jcking1034 updated this revision to Diff 386577.
jcking1034 added a comment.
Add additional unit tests.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113575/new/
https://reviews.llvm.org/D113575
Files:
clang/docs/LibASTMatchersReference.html
c
fowles added inline comments.
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:4224
+/// float z;
+/// auto f = [=]() { return x + y + z; };
+/// }
it would be nice to be able to do something like
```
int main() {
int x, y;
float z;
auto
jcking1034 created this revision.
jcking1034 added reviewers: ymandel, tdl-g, aaron.ballman, fowles.
jcking1034 requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This contributes follow-up work from https://reviews.llvm.org/D112491, which
all
