Fix a braino in JitCodeCache::RemoveMethodsIn.

Only update `it` only after using it. Otherwise we're using the wrong
element, or using `end()`.

Spotted by mingaleev@!

Test: test.py
Change-Id: Ieb848b8f28f6d53abde6c3058b6cd2d4b1c79a1b

1 files changed, 1 insertions, 1 deletions

diff --git a/runtime/jit/jit_code_cache.cc b/runtime/jit/jit_code_cache.cc
index e6db9817df..89be345875 100644
--- a/runtime/jit/jit_code_cache.cc
+++ b/runtime/jit/jit_code_cache.cc
@@ -567,9 +567,9 @@ void JitCodeCache::RemoveMethodsIn(Thread* self, const LinearAlloc& alloc) {
       if (alloc.ContainsUnsafe(it->second)) {
         method_headers.insert(OatQuickMethodHeader::FromCodePointer(it->first));
         VLOG(jit) << "JIT removed " << it->second->PrettyMethod() << ": " << it->first;
-        it = method_code_map_.erase(it);
         zombie_code_.erase(it->first);
         processed_zombie_code_.erase(it->first);
+        it = method_code_map_.erase(it);
       } else {
         ++it;
       }