ilya-biryukov wrote:

> Yeah, I think this patch may be conceptually good except the extra memory 
> using. But benchmarking, I tried it locally but didn't find observable 
> effects. So I'd like to land this after 19's branching to give it more baking 
> time.

Do you mean that no large increases in the memory use or that things are not 
getting faster either?
If it's the latter, I would vouch for keeping the code as-is to avoid adding 
extra complexity.
If this does lead to significant performance wins, though, it's a different 
story.

https://github.com/llvm/llvm-project/pull/95506
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to