[llvm-branch-commits] [clang] [Clang] [C++26] Expansion Statements (Part 10: Expansion Limit) (PR #169689)

[via llvm-branch-commits]

Sirraide wrote:

> Is an actual performance concern here?
> 
> Unlike recursive things it's bound to terminate, and it's not adding recursion in itself - the limit from the compiler perspective is the number of instantiations / objects in the declaration context.
> 
> I expect people will want to iterate over the member of `::` or `::std` and expect that to somehow work.
> 
> So i think this is fine for now but we will probably have to adjust with use cases

Iirc when I was testing this it did 1 million expansions w/o hitting any existing limits; that was w/ an empty expansion statement but that already took a few seconds.

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