AaronBallman wrote:

> We have Clang LGTM from @AaronBallman and LLVM LGTM from @bwendling. I've 
> rebased and squashed to keep this up-to-date. Are we ready to go on this? Are 
> people okay with the ~1% memory overhead induced from this feature being 
> on-by-default? We can always add a flag if this is a non-starter.

I think ~1% memory overhead is acceptable given the benefits but I've added 
@nikic to the review for more opinions from a different perspective.

https://github.com/llvm/llvm-project/pull/174892
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to