rjmccall added a comment.

I agree with Richard's suggestion of just numbering all the lambdas in both 
modes if that's viable.



================
Comment at: clang/include/clang/AST/DeclCXX.h:587-590
+    unsigned NumExplicitCaptures : 12;
+
+    /// Has known `internal` linkage.
+    unsigned HasKnownInternalLinkage : 1;
----------------
hliao wrote:
> Try to avoid inflating memory footprint by borrowing one bit from 
> `NumExplicitCaptures`. It seems to me that we won't have that large number 
> (8192) of *explicit* capture.
If this field is necessary, please borrow its space from `ManglingNumber` below 
rather than the number of explicit captures.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68818



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

Reply via email to