[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