aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM aside from a minor nit.
Comment at: clang/lib/AST/Decl.cpp:2130
+ // the pretty-printed name of the capture instead.
+ if (isa(DD) &&
+ maybePrintFi
riccibruno updated this revision to Diff 282391.
riccibruno edited the summary of this revision.
riccibruno added a comment.
Don't forget to increment the field iterator in the loop of
`maybePrintFieldForLambdaCapture`, and modify the tests to test this.
Repository:
rG LLVM Github Monorepo
C
riccibruno updated this revision to Diff 282312.
riccibruno added a comment.
Add `-fno-delayed-template-parsing` to the new unit tests to also pass on
Windows.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85033/new/
https://reviews.llvm.org/D8503
riccibruno updated this revision to Diff 282296.
riccibruno added a comment.
Make the unit tests in `NamedDeclPrinterTest.cpp` more robust.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85033/new/
https://reviews.llvm.org/D85033
Files:
clang/lib