suyu/dynarmic
28
1
Fork 1
Code Issues Pull requests Projects Releases Packages Wiki Activity

A32/Thumb: Correct behaviour for UDF and Unpredictable instructions

Browse source 
Raise an exception instead of calling the interpreter and ASSERT-ing respectively.
This commit is contained in:
MerryMage 2019-07-27 19:56:18 +01:00
parent c7d20f3f2f
commit 1aa7b62e92
4 changed files with 12 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/frontend/A32/translate/impl/thumb16.cpp
View file

@ -885,7 +885,7 @@ bool ThumbTranslatorVisitor::thumb16_CBZ_CBNZ(bool nonzero, Imm<1> i, Imm<5> imm
}
bool ThumbTranslatorVisitor::thumb16_UDF() {
return InterpretThisInstruction();
return UndefinedInstruction();
}
// BX <Rm>

1
src/frontend/A32/translate/impl/translate_thumb.h
View file

@ -29,6 +29,7 @@ struct ThumbTranslatorVisitor final {
bool InterpretThisInstruction();
bool UnpredictableInstruction();
bool UndefinedInstruction();
bool RaiseException(Exception exception);
// thumb16

11
src/frontend/A32/translate/translate_thumb.cpp
View file

@ -122,12 +122,19 @@ bool ThumbTranslatorVisitor::InterpretThisInstruction() {
}
bool ThumbTranslatorVisitor::UnpredictableInstruction() {
ASSERT_MSG(false, "UNPREDICTABLE");
ir.ExceptionRaised(Exception::UnpredictableInstruction);
ir.SetTerm(IR::Term::CheckHalt{IR::Term::ReturnToDispatch{}});
return false;
}
bool ThumbTranslatorVisitor::UndefinedInstruction() {
ir.ExceptionRaised(Exception::UndefinedInstruction);
ir.SetTerm(IR::Term::CheckHalt{IR::Term::ReturnToDispatch{}});
return false;
}
bool ThumbTranslatorVisitor::RaiseException(Exception exception) {
ir.BranchWritePC(ir.Imm32(ir.current_location.PC() + 2));
ir.BranchWritePC(ir.Imm32(ir.current_location.PC() + 2)); // TODO: T32
ir.ExceptionRaised(exception);
ir.SetTerm(IR::Term::CheckHalt{IR::Term::ReturnToDispatch{}});
return false;

2
tests/A32/testenv.h
View file

@ -78,7 +78,7 @@ public:
MemoryWrite32(vaddr + 4, static_cast<u32>(value >> 32));
}
void InterpreterFallback(u32 pc, size_t num_instructions) override { ASSERT_MSG(false, "InterpreterFallback({:08x}, {})", pc, num_instructions); }
void InterpreterFallback(u32 pc, size_t num_instructions) override { ASSERT_MSG(false, "InterpreterFallback({:08x}, {}) code = {:08x}", pc, num_instructions, MemoryReadCode(pc)); }
void CallSVC(std::uint32_t swi) override { ASSERT_MSG(false, "CallSVC({})", swi); }