get rid of std::oldmap (migrate to core::hashmap) #4986
Description
This is at least 30% slower than core::hashmap, not sendable, and it doesn't use explicit self or expose the underlying mutability through the API (relies on mut fields). It also results in a lot of code bloat and is probably responsible for slowing down the compiler by a significant amount.
