rsmith added inline comments.

================
Comment at: clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp:485
+  A a;
+  auto l = [a] { };
+  auto lCopy = l;
----------------
steveire wrote:
> rsmith wrote:
> > steveire wrote:
> > > I don't know how to create a lambda with a default ctor body.
> > I think that's probably not actually possible, sorry for misleading you on 
> > that! You can introduce a copy constructor with a body (by giving `A` a 
> > non-trivial copy constructor), though, if you want to test that.
> Thanks! I think I have the copy-ctor case covered already. Is something 
> missing?
Oh, right you are. Looks good :)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95644/new/

https://reviews.llvm.org/D95644

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

Reply via email to