gas: unhelpful sprint_value() behavior

[Jan Beulich]

Hello,

I've just run into a testsuite failure (from an addition I have pending)
resulting from sprint_value() printing a value in either hex or decimal
depending on sizeof(long) vs sizeof(bfd_vma). Large values (particularly
on, but not limited to, 64-bit architectures) are imo much harder to
make sense of when presented in decimal.

Since addressT, valueT, and bfd_vma are all the same (nowadays?) anyway
and since there are pretty few uses of the function, would there be any
objection to dropping it, making the callers use sprint_vma() instead?

Thanks, Jan