Clear the symbol map between each test.

This way we could, for example, test function replacement properly.

Actually slows down tests because it spends more time correctly
determining the function boundaries.
Spend a bit less time in Read_Instruction on load.
Speed up MIPSGetInstruction() by 35%.

It's not that hot, but gets called a lot when loading modules / running
tests.  Well, and also interpreter.

Some of the previous checks were just not necessary (we never have Rese in
the tables, for example, and let's just not.)
Cut down on locking in DEBUG_LOG.

Reduces GenericLog() from 13.8% to 1.5% in Where Is My Heart?, will
probably help other games to a lesser extent.
Merge pull request #5585 from unknownbrackets/perf

Improve performance when running tests / fix function replacement