get rid of std::oldmap (migrate to core::hashmap)

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.
