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



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

Reply via email to