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
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to