[PATCH] D154716: [SemaCXX] Fix bug where unexpanded lambda captures where assumed to be expanded

Mital Ashok via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jul 22 07:07:47 PDT 2023


MitalAshok marked an inline comment as done.
MitalAshok added inline comments.


================
Comment at: clang/lib/Sema/TreeTransform.h:13323
 
     // If this is an init-capture pack, consider expanding the pack now.
     if (OldVD->isParameterPack()) {
----------------
shafik wrote:
> Based on the changes is this comment still accurate?
Yes, the code was just changed so it's more clear that the result either has `N` distinct arguments for a pack of size `N` or just 1 argument with a (possible dependent) pack type


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154716



More information about the cfe-commits mailing list