[clang] [clang][NFC] Increase NumStmtBits by 2 as we are approaching the limit (PR #120341)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 18 08:22:56 PST 2024


erichkeane wrote:

> Let's just increase to 9 bits.
> 
> Have you checked whether the size of `Stmt` or `Expr` changes when you do this?

I have these concerns as well, this ends up affecting the size of a lot of things....

That said, StmtNodes.td already has 267 items in it (as far as I can tell, just a wc -l on appearances of 'def').  So I don't know how we're getting away with this already.

Can you confirm how many are present currently? 

I also think 9 should be plenty, 512 should cover us for many many years.

https://github.com/llvm/llvm-project/pull/120341


More information about the cfe-commits mailing list