[PATCH] D50713: [AST] Pack the unsigned of SubstTemplateTypeParmPackType into Type

Bruno Ricci via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 15 06:16:21 PDT 2018


riccibruno added a comment.

The thing is that I have no idea what is the minimum number
of bits required. It was originally an unsigned but I suspect that
32 bits are not needed. Looking at [implimits] I see

  Template arguments in a template declaration [1 024].
  Recursively nested template instantiations, including substitution during template argument deduc-
  tion (17.8.2) [1 024].

But 1024 seems to be too small and easily exceeded by a doing a bit
of template meta-programming.


Repository:
  rC Clang

https://reviews.llvm.org/D50713





More information about the cfe-commits mailing list