Correct the clearing/destruction order when switching JITs.

Fixes #20502

Author: hrydgard

Core/MIPS/IR/IRJit.h

@@ -119,6 +119,9 @@ class IRBlock {
 class IRBlockCache : public JitBlockCacheDebugInterface {
 public:
 	IRBlockCache(bool compileToNative);
+	~IRBlockCache() {
+		Clear();
+	}
 
 	void Clear();
 	std::vector<int> FindInvalidatedBlockNumbers(u32 address, u32 length);

Core/MIPS/MIPS.cpp

@@ -228,39 +228,34 @@ void MIPSState::UpdateCore(CPUCore desired) {
 		return;
 	}
 
+	if (MIPSComp::jit) {
+		std::lock_guard<std::recursive_mutex> guard(MIPSComp::jitLock);
+		delete MIPSComp::jit;
+		MIPSComp::jit = nullptr;
+	}
+
 	PSP_CoreParameter().cpuCore = desired;
-	MIPSComp::JitInterface *oldjit = MIPSComp::jit;
 	MIPSComp::JitInterface *newjit = nullptr;
 
 	switch (PSP_CoreParameter().cpuCore) {
 	case CPUCore::JIT:
 	case CPUCore::JIT_IR:
 		INFO_LOG(Log::CPU, "Switching to JIT%s", PSP_CoreParameter().cpuCore == CPUCore::JIT_IR ? " IR" : "");
-		if (oldjit) {
-			std::lock_guard<std::recursive_mutex> guard(MIPSComp::jitLock);
-			MIPSComp::jit = nullptr;
-			delete oldjit;
-		}
 		newjit = MIPSComp::CreateNativeJit(this, PSP_CoreParameter().cpuCore == CPUCore::JIT_IR);
 		break;
 
 	case CPUCore::IR_INTERPRETER:
 		INFO_LOG(Log::CPU, "Switching to IR interpreter");
-		if (oldjit) {
-			std::lock_guard<std::recursive_mutex> guard(MIPSComp::jitLock);
-			MIPSComp::jit = nullptr;
-			delete oldjit;
-		}
 		newjit = new MIPSComp::IRJit(this, false);
 		break;
 
 	case CPUCore::INTERPRETER:
 		INFO_LOG(Log::CPU, "Switching to interpreter");
-		if (oldjit) {
-			std::lock_guard<std::recursive_mutex> guard(MIPSComp::jitLock);
-			MIPSComp::jit = nullptr;
-			delete oldjit;
-		}
+		break;
+
+	default:
+		WARN_LOG(Log::CPU, "Invalid value for cpuCore, falling back to interpreter");
+		// Leave as nullptr.
 		break;
 	}