DebugInfo: Don't create duplicate forward declaration metadata unnecessarily.

No functionality change, at best a slight (questionable) optimization,
but necessary for correctness of future work.
DebugInfo: remove unnecessary type registration

This happens in the caller a few frames up anyway.