kbobyrev added inline comments.

================
Comment at: clang/lib/Sema/SemaExpr.cpp:17457
+  SmallString<32> FixBuffer;
+  StringRef Separator = LSI->NumExplicitCaptures > 0 ? ", " : "";
+  if (Var->getDeclName().isIdentifier() && !Var->getName().empty()) {
----------------
sammccall wrote:
> This logic assumes there's no default capture, whic his reasonable but not 
> locally obvious - add an assert?
I think this is the same as https://reviews.llvm.org/D96975#inline-920917 
(which is done), am I wrong?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96975

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

Reply via email to