[libcxx-commits] [libcxx] [libc++] Make the body of println(FILE*) dependent on the template parameter to avoid template instantiation (PR #200996)

[Braden Ganetsky via libcxx-commits]

k3DW wrote:

> I'm not convinced this is a good idea as-is. The `template <class = void>` is meant to be there temporarily, and this patch makes it basically a permanent requirement.

The template head is there temporarily for what end? What would it ultimately be replaced with?

I also don't see how this makes it permanently required. If the template parameter is removed, then the body of the function can be changed back. There is no effect on ABI or API boundaries here, so it can be freely changed whenever needed.

Again though, I'm happy with any solution that avoids instantiating all those templates I mentioned in the original issue. This can be reverted in favour of something better. I just don't want all those function template instantiations.

https://github.com/llvm/llvm-project/pull/200996