make `fn_sig().subst()` ICE when used with a closure

make `fn_sig().subst()` ICE when used with a closure

It's inefficient, and the substitution there doesn't account for the
extra regions used by NLL inference, so it's a bad thing to encourage.

As it happens all callers already know if they have a closure or not,
from what I can tell.