https://github.com/steakhal commented:

This patch looks good to me.

IMO we should not rewrite unused templates - because they are unused anyway, so 
why bother.

Template instantiation will be represented in the current implementation, and 
assuming complete whole-program analyses, all users (instantiations) of the 
templates will be accounted for. Consequently, I have no correctness concerns.

I only had minor suggestions.

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

Reply via email to