[PATCH] D20732: Don't use static variables in LambdaCapture
James Y Knight via cfe-commits
cfe-commits at lists.llvm.org
Mon Jun 13 09:17:07 PDT 2016
jyknight added a comment.
This looks okay. The only downside: it will increase sizeof(Decl) on 32bit platforms, since the layout is:
vtable ptr (4)
NextInContextAndBits ptr (4)
DeclCtx ptr (4)
Loc int (4)
bitfield int (4)
Totaling 20 bytes, which will now be rounded up to 24 bytes. I don't know how others feel, but it seems probably fine to me? On 64-bit platforms, it's already 32 bytes, and that won't be affected.
Repository:
rL LLVM
http://reviews.llvm.org/D20732
More information about the cfe-commits
mailing list