ChuanqiXu9 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.

Yeah, it is the later. But it might not be too complex since all its uses are 
limited to the file only. My point is on the contrast, if it doesn't lead to 
significant regression, we should do it. Since it is rare that a single patch 
did herotical optimizations. Some times the performance gains from the long 
tails.

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