From 967c371d7a8695e60cf52da22f3952f23b031359 Mon Sep 17 00:00:00 2001 From: Scott Mansell Date: Sun, 28 Aug 2016 09:48:29 +1200 Subject: [PATCH] Zero case behaviour confirmed by hardware test Modify comments to reflect this --- Source/Core/Core/PowerPC/Interpreter/Interpreter_LoadStore.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Source/Core/Core/PowerPC/Interpreter/Interpreter_LoadStore.cpp b/Source/Core/Core/PowerPC/Interpreter/Interpreter_LoadStore.cpp index 0df1c338b5..e3e1f4fcd6 100644 --- a/Source/Core/Core/PowerPC/Interpreter/Interpreter_LoadStore.cpp +++ b/Source/Core/Core/PowerPC/Interpreter/Interpreter_LoadStore.cpp @@ -497,7 +497,6 @@ void Interpreter::lhzx(UGeckoInstruction _inst) } } -// TODO: is this right? // FIXME: Should rollback if a DSI occurs void Interpreter::lswx(UGeckoInstruction _inst) { @@ -506,6 +505,7 @@ void Interpreter::lswx(UGeckoInstruction _inst) int r = _inst.RD; int i = 0; + // Confirmed by hardware test that the zero case doesn't zero rGPR[r] if (n > 0) { rGPR[r] = 0;