Consider moving DebugInfo out of noirc_errors

[TomAFrench]

It doesn't really sit right with me that we're declaring the format for how we embed debug information into build artifacts into the crate which just defines how we print out error messages.

I think it makes more sense for the DebugInfo struct to be defined along with the rest of the compiler's build artifacts. This would allow us to simplify the dependency graph a bit (noirc_errors should not need to know about the ACVM).