aboutsummaryrefslogtreecommitdiff
path: root/src/dynarmic/tests
diff options
context:
space:
mode:
authorlizzie <lizzie@eden-emu.dev>2026-09-19 01:39:46 +0200
committercrueter <crueter@eden-emu.dev>2026-09-19 01:39:46 +0200
commit9a30172e45fcb7833cc49ba4d6450211449b14df (patch)
tree4ab0ffbb276464086615fa247d69a9cdf9d5ac55 /src/dynarmic/tests
parent07f40d5cac635ff102aa6ce9c32572cc20d69959 (diff)
[dynarmic] Coalesce non-exclusive Write/Read fallback functions (#4158)
the general idea is to have a common procedure form whom to call this way theres less "jumping around" for values of different sizes additionally this **should** allow for better codegen since most of the u8,u16,u32,u64 can be held within a u64 theoretically this means that you could deifne callbacks in suck a way that it's essentially as costly as a `mov r64, m64` but that's not doable due to the fact we have to do translations... Is this a good change? Primarily aimed for x86 and ARM to benefit, but I suppose LooooooongArch64 can benefit too. Signed-off-by: lizzie <lizzie@eden-emu.dev> Reviewed-on: https://git.eden-emu.dev/eden-emu/eden/pulls/4158 Reviewed-by: crueter <crueter@eden-emu.dev> Reviewed-by: MaranBr <maranbr@eden-emu.dev> Reviewed-by: CamilleLaVey <camillelavey99@gmail.com>
Diffstat (limited to 'src/dynarmic/tests')
-rw-r--r--src/dynarmic/tests/A32/fuzz_thumb.cpp2
-rw-r--r--src/dynarmic/tests/A32/test_arm_instructions.cpp6
-rw-r--r--src/dynarmic/tests/A32/testenv.h127
-rw-r--r--src/dynarmic/tests/A64/a64.cpp20
-rw-r--r--src/dynarmic/tests/A64/fibonacci.cpp85
-rw-r--r--src/dynarmic/tests/A64/testenv.h142
-rw-r--r--src/dynarmic/tests/print_info.cpp72
-rw-r--r--src/dynarmic/tests/unicorn_emu/a32_unicorn.cpp21
-rw-r--r--src/dynarmic/tests/unicorn_emu/a64_unicorn.cpp21
9 files changed, 248 insertions, 248 deletions
diff --git a/src/dynarmic/tests/A32/fuzz_thumb.cpp b/src/dynarmic/tests/A32/fuzz_thumb.cpp
index 67a01daf9c..a29b877f3e 100644
--- a/src/dynarmic/tests/A32/fuzz_thumb.cpp
+++ b/src/dynarmic/tests/A32/fuzz_thumb.cpp
@@ -503,7 +503,7 @@ TEST_CASE("Fuzz Thumb32 instructions set", "[JitX64][Thumb][Thumb32]") {
}
}
-TEST_CASE("Verify fix for off by one error in MemoryRead32 worked", "[Thumb][Thumb16]") {
+TEST_CASE("Verify fix for off by one error in MemoryRead<32> worked", "[Thumb][Thumb16]") {
ThumbTestEnv test_env;
// Prepare test subjects
diff --git a/src/dynarmic/tests/A32/test_arm_instructions.cpp b/src/dynarmic/tests/A32/test_arm_instructions.cpp
index 5a27cd499c..ee24131386 100644
--- a/src/dynarmic/tests/A32/test_arm_instructions.cpp
+++ b/src/dynarmic/tests/A32/test_arm_instructions.cpp
@@ -553,9 +553,9 @@ TEST_CASE("arm: Memory access (fastmem)", "[arm][A32]") {
memset(backing_memory, 0, memory_size);
memcpy(backing_memory + 0x100, "Lorem ipsum dolor sit amet, consectetur adipiscing elit.", 57);
- env.MemoryWrite32(0, 0xE5904000); // LDR R4, [R0]
- env.MemoryWrite32(4, 0xE5814000); // STR R4, [R1]
- env.MemoryWrite32(8, 0xEAFFFFFE); // B .
+ env.MemoryWrite(0, 0xE5904000, sizeof(u32)); // LDR R4, [R0]
+ env.MemoryWrite(4, 0xE5814000, sizeof(u32)); // STR R4, [R1]
+ env.MemoryWrite(8, 0xEAFFFFFE, sizeof(u32)); // B .
jit.Regs()[0] = 0x100;
jit.Regs()[1] = 0x1F0;
jit.Regs()[15] = 0; // PC = 0
diff --git a/src/dynarmic/tests/A32/testenv.h b/src/dynarmic/tests/A32/testenv.h
index bebc2566da..a22cc7131e 100644
--- a/src/dynarmic/tests/A32/testenv.h
+++ b/src/dynarmic/tests/A32/testenv.h
@@ -16,6 +16,7 @@
#include "common/assert.h"
#include "common/common_types.h"
+#include "dynarmic/frontend/A32/translate/translate_callbacks.h"
#include "dynarmic/interface/A32/a32.h"
template<typename InstructionType_, u32 infinite_loop_u32>
@@ -59,42 +60,51 @@ public:
return infinite_loop_u32; // B .
}
- std::uint8_t MemoryRead8(u32 vaddr) override {
- if (IsInCodeMem(vaddr)) {
- return reinterpret_cast<u8*>(code_mem.data())[vaddr];
+ u64 MemoryRead(u32 vaddr, size_t size) override {
+ switch (size) {
+ case sizeof(u64):
+ return MemoryRead(vaddr, sizeof(u32))
+ | MemoryRead(vaddr + sizeof(u32), sizeof(u32)) << 32;
+ case sizeof(u32):
+ return MemoryRead(vaddr, sizeof(u16))
+ | MemoryRead(vaddr + sizeof(u16), sizeof(u16)) << 16;
+ case sizeof(u16):
+ return MemoryRead(vaddr, sizeof(u8))
+ | MemoryRead(vaddr + sizeof(u8), sizeof(u8)) << 8;
+ case sizeof(u8): {
+ if (IsInCodeMem(vaddr))
+ return reinterpret_cast<u8*>(code_mem.data())[vaddr];
+ if (auto iter = modified_memory.find(vaddr); iter != modified_memory.end())
+ return iter->second;
+ return u8(vaddr);
}
- if (auto iter = modified_memory.find(vaddr); iter != modified_memory.end()) {
- return iter->second;
+ default:
+ std::abort();
}
- return static_cast<u8>(vaddr);
- }
- std::uint16_t MemoryRead16(u32 vaddr) override {
- return u16(MemoryRead8(vaddr)) | u16(MemoryRead8(vaddr + 1)) << 8;
- }
- std::uint32_t MemoryRead32(u32 vaddr) override {
- return u32(MemoryRead16(vaddr)) | u32(MemoryRead16(vaddr + 2)) << 16;
- }
- std::uint64_t MemoryRead64(u32 vaddr) override {
- return u64(MemoryRead32(vaddr)) | u64(MemoryRead32(vaddr + 4)) << 32;
}
- void MemoryWrite8(u32 vaddr, std::uint8_t value) override {
- if (vaddr < code_mem.size() * sizeof(u32)) {
- code_mem_modified_by_guest = true;
+ void MemoryWrite(Dynarmic::A32::VAddr vaddr, u64 value, size_t size) override {
+ switch (size) {
+ case sizeof(u64):
+ MemoryWrite(vaddr, u32(value), sizeof(u32));
+ MemoryWrite(vaddr + 4, u32(value >> 32), sizeof(u32));
+ break;
+ case sizeof(u32):
+ MemoryWrite(vaddr, u16(value), sizeof(u16));
+ MemoryWrite(vaddr + 2, u16(value >> 16), sizeof(u16));
+ break;
+ case sizeof(u16):
+ MemoryWrite(vaddr, u8(value), sizeof(u8));
+ MemoryWrite(vaddr + 1, u8(value >> 8), sizeof(u8));
+ break;
+ case sizeof(u8):
+ if (vaddr < code_mem.size() * sizeof(u32))
+ code_mem_modified_by_guest = true;
+ modified_memory[vaddr] = value;
+ break;
+ default:
+ std::abort();
}
- modified_memory[vaddr] = value;
- }
- void MemoryWrite16(u32 vaddr, std::uint16_t value) override {
- MemoryWrite8(vaddr, static_cast<u8>(value));
- MemoryWrite8(vaddr + 1, static_cast<u8>(value >> 8));
- }
- void MemoryWrite32(u32 vaddr, std::uint32_t value) override {
- MemoryWrite16(vaddr, static_cast<u16>(value));
- MemoryWrite16(vaddr + 2, static_cast<u16>(value >> 16));
- }
- void MemoryWrite64(u32 vaddr, std::uint64_t value) override {
- MemoryWrite32(vaddr, static_cast<u32>(value));
- MemoryWrite32(vaddr + 4, static_cast<u32>(value >> 32));
}
void CallSVC(std::uint32_t swi) override {
@@ -143,46 +153,41 @@ public:
return read<std::uint32_t>(vaddr);
}
- std::uint8_t MemoryRead8(std::uint32_t vaddr) override {
- return read<std::uint8_t>(vaddr);
- }
- std::uint16_t MemoryRead16(std::uint32_t vaddr) override {
- return read<std::uint16_t>(vaddr);
- }
- std::uint32_t MemoryRead32(std::uint32_t vaddr) override {
- return read<std::uint32_t>(vaddr);
- }
- std::uint64_t MemoryRead64(std::uint32_t vaddr) override {
- return read<std::uint64_t>(vaddr);
+ u64 MemoryRead(u32 vaddr, size_t size) override {
+ switch (size) {
+ case sizeof(u64): return read<u64>(vaddr);
+ case sizeof(u32): return read<u32>(vaddr);
+ case sizeof(u16): return read<u16>(vaddr);
+ case sizeof(u8): return read<u8>(vaddr);
+ default:
+ std::abort();
+ }
}
- void MemoryWrite8(std::uint32_t vaddr, std::uint8_t value) override {
- write(vaddr, value);
- }
- void MemoryWrite16(std::uint32_t vaddr, std::uint16_t value) override {
- write(vaddr, value);
- }
- void MemoryWrite32(std::uint32_t vaddr, std::uint32_t value) override {
- write(vaddr, value);
- }
- void MemoryWrite64(std::uint32_t vaddr, std::uint64_t value) override {
- write(vaddr, value);
+ void MemoryWrite(Dynarmic::A32::VAddr vaddr, std::uint64_t value, size_t size) override {
+ switch (size) {
+ case sizeof(u64): return write<u64>(vaddr, u64(value));
+ case sizeof(u32): return write<u32>(vaddr, u32(value));
+ case sizeof(u16): return write<u16>(vaddr, u16(value));
+ case sizeof(u8): return write<u8>(vaddr, u8(value));
+ default: std::abort();
+ }
}
- bool MemoryWriteExclusive8(std::uint32_t vaddr, std::uint8_t value, [[maybe_unused]] std::uint8_t expected) override {
- MemoryWrite8(vaddr, value);
+ bool MemoryWriteExclusive8(Dynarmic::A32::VAddr vaddr, std::uint8_t value, [[maybe_unused]] std::uint8_t expected) override {
+ MemoryWrite(vaddr, value, sizeof(u8));
return true;
}
- bool MemoryWriteExclusive16(std::uint32_t vaddr, std::uint16_t value, [[maybe_unused]] std::uint16_t expected) override {
- MemoryWrite16(vaddr, value);
+ bool MemoryWriteExclusive16(Dynarmic::A32::VAddr vaddr, std::uint16_t value, [[maybe_unused]] std::uint16_t expected) override {
+ MemoryWrite(vaddr, value, sizeof(u16));
return true;
}
- bool MemoryWriteExclusive32(std::uint32_t vaddr, std::uint32_t value, [[maybe_unused]] std::uint32_t expected) override {
- MemoryWrite32(vaddr, value);
+ bool MemoryWriteExclusive32(Dynarmic::A32::VAddr vaddr, std::uint32_t value, [[maybe_unused]] std::uint32_t expected) override {
+ MemoryWrite(vaddr, value, sizeof(u32));
return true;
}
- bool MemoryWriteExclusive64(std::uint32_t vaddr, std::uint64_t value, [[maybe_unused]] std::uint64_t expected) override {
- MemoryWrite64(vaddr, value);
+ bool MemoryWriteExclusive64(Dynarmic::A32::VAddr vaddr, std::uint64_t value, [[maybe_unused]] std::uint64_t expected) override {
+ MemoryWrite(vaddr, value, sizeof(u64));
return true;
}
diff --git a/src/dynarmic/tests/A64/a64.cpp b/src/dynarmic/tests/A64/a64.cpp
index 99cc9d3d4d..24ddfd4084 100644
--- a/src/dynarmic/tests/A64/a64.cpp
+++ b/src/dynarmic/tests/A64/a64.cpp
@@ -1006,8 +1006,8 @@ TEST_CASE("A64: 128-bit exclusive read/write", "[a64]") {
REQUIRE(jit.GetRegister(1) == 0x7f7e7d7c7b7a7978);
REQUIRE(jit.GetRegister(2) == 0x8786858483828180);
REQUIRE(jit.GetRegister(4) == 0);
- REQUIRE(env.MemoryRead64(0x1234567812345678) == 0xaf00d1e5badcafe0);
- REQUIRE(env.MemoryRead64(0x1234567812345680) == 0xd0d0cacad0d0caca);
+ REQUIRE(env.MemoryRead(0x1234567812345678, sizeof(u64)) == 0xaf00d1e5badcafe0);
+ REQUIRE(env.MemoryRead(0x1234567812345680, sizeof(u64)) == 0xd0d0cacad0d0caca);
}
TEST_CASE("A64: CNTPCT_EL0", "[a64]") {
@@ -1517,11 +1517,11 @@ TEST_CASE("A64: Memory access (fastmem)", "[a64]") {
memset(backing_memory, 0, memory_size);
memcpy(backing_memory + 0x100, "Lorem ipsum dolor sit amet, consectetur adipiscing elit.", 57);
- env.MemoryWrite32(0, 0xA9401404); // LDP X4, X5, [X0]
- env.MemoryWrite32(4, 0xF9400046); // LDR X6, [X2]
- env.MemoryWrite32(8, 0xA9001424); // STP X4, X5, [X1]
- env.MemoryWrite32(12, 0xF9000066); // STR X6, [X3]
- env.MemoryWrite32(16, 0x14000000); // B .
+ env.MemoryWrite(0, 0xA9401404, sizeof(u32)); // LDP X4, X5, [X0]
+ env.MemoryWrite(4, 0xF9400046, sizeof(u32)); // LDR X6, [X2]
+ env.MemoryWrite(8, 0xA9001424, sizeof(u32)); // STP X4, X5, [X1]
+ env.MemoryWrite(12, 0xF9000066, sizeof(u32)); // STR X6, [X3]
+ env.MemoryWrite(16, 0x14000000, sizeof(u32)); // B .
jit.SetRegister(0, 0x100);
jit.SetRegister(1, 0x1F0);
jit.SetRegister(2, 0x10F);
@@ -1788,8 +1788,8 @@ TEST_CASE("A64: rand3", "[a64]") {
memset(backing_memory, 0, memory_size);
// cat rand2.txt | awk '{print "env.code_mem.emplace_back(0x"$2"); // "$0}' > rand2-out.tx
- env.MemoryWrite32(100, 0x58028edd); // 0000000000000084 58028edd ldr x29, #20952
- env.MemoryWrite32(104, 0x14000000); // 0000000000000ea4 14000000 b #0
+ env.MemoryWrite(100, 0x58028edd, sizeof(u32)); // 0000000000000084 58028edd ldr x29, #20952
+ env.MemoryWrite(104, 0x14000000, sizeof(u32)); // 0000000000000ea4 14000000 b #0
jit.SetPC(100);
jit.SetPstate(0xb0000000);
@@ -1824,7 +1824,7 @@ TEST_CASE("A64: rand2", "[a64][.]") {
// cat rand2.txt | awk '{print "env.code_mem.emplace_back(0x"$2"); // "$0}' > rand2-out.tx
const std::array<uint32_t, 4096> code32 = {0xea80f352, 0x6e65e59d, 0x1e20c343, 0x2e3a7192, 0x2e267249, 0xd500405f, 0x6f01f461, 0x6eb684fc, 0x58028edd, 0x0ea5f5b6, 0x0ea069fb, 0x2e769517, 0x5e066063, 0x1e65c3f5, 0x4f00ff52, 0x93401cf6, 0x1e274248, 0x6f67aaf5, 0x5e0c0782, 0x5ef43f3c, 0x2e6595b7, 0x4e20590f, 0xb35aa451, 0x6ee2c5ed, 0x4e32bf46, 0x2ea1ba8f, 0x2f68a85e, 0x9237d90a, 0x5e23dd10, 0x0e762e32, 0x4e31a8cf, 0xce1f3360, 0x781a4ac0, 0x13834066, 0x5fa8101c, 0x6f7c5594, 0x0e71bb68, 0xbc0b3e8f, 0x785dbbda, 0x6f51e794, 0xce50af75, 0x1ad728ec, 0x6ee0da4c, 0xb84efa14, 0x2eb3f613, 0x4e287ade, 0x4eb8c734, 0x2e83f4e8, 0x0e397c80, 0xd08f93f8, 0xce718e48, 0x0f672a0d, 0x2e9edd40, 0x0e14128b, 0x6f5942e6, 0x8b3a0f03, 0x3c5d16b9, 0x7f7e3743, 0x4f4c54e4, 0x0ea0a9e9, 0x9e59dbe6, 0x6e7ddcd3, 0xcec08377, 0x9ba759f8, 0x2ea5046e, 0x0e24c569, 0xb8979780, 0x4e31b98c, 0x4efe4f46, 0x4ea7c762, 0x7e61c9c6, 0x6e30c880, 0x1ada0c25, 0x4e603a2f, 0xda9d7218, 0x0d40c5d9, 0x5e214b05, 0x9ba9efc5, 0x5e61b81e, 0x6e7bc31c, 0x0e61a163, 0x9e5832d2, 0x4e772248, 0x4e3d17c8, 0x92624f60, 0x7a1a02dc, 0x79891f65, 0x6eb45036, 0x0e321ee8, 0x4e2566f0, 0x4ea02b9b, 0x0f9dcb3d, 0x2e21b9f9, 0x0e21a8c3, 0xda1700bd, 0x6ea0fb38, 0x7e607a0b, 0x72845817, 0x7f61068e, 0x0d60e529, 0x4ea0ca5c, 0x1a94b20f, 0x8b87419d, 0x7ea9ed71, 0x2ea1a86e, 0x4d40c4da, 0x5ea0eada, 0x784ba96e, 0x7eb6ee02, 0x3db1c710, 0x0e217836, 0x7ee0bb96, 0x4e786c08, 0x4e976a08, 0x489ffe86, 0x4e79fc9b, 0x0e21cbce, 0x5ef7fc65, 0x4ea1286d, 0xd29c771e, 0x6f5c2839, 0x0ea00a9d, 0x6ee44c06, 0x5ee1d858, 0x5ef2fda6, 0x7eb0c9fe, 0x7f762791, 0x2e212ae6, 0x4e61c9db, 0x13003c57, 0x5ee1b8f8, 0x0f2396d2, 0x6ea0db1e, 0x0e71ba82, 0xab29c807, 0x6ef8f8b3, 0x1f18d4a1, 0x0e261d15, 0x1e290081, 0x1b0c7d12, 0x4e7771c3, 0xf845f1e4, 0x4d40c9e8, 0xce778452, 0x6eb9879d, 0x6e21c93d, 0xcec0829f, 0x52a0969f, 0x1e772b4f, 0x7ee1da88, 0x5f52fe0a, 0x7f3387b1, 0x5e214850, 0x1e65c025, 0x0e2ca294, 0x2e614829, 0x1e640077, 0x9e240048, 0x4ebe9537, 0x9bb7925e, 0x38b669c5, 0x2840d089, 0x6f43e648, 0x2e662d28, 0x4eabaff3, 0x6e734cc7, 0x0e31baee, 0x7ee0d93c, 0x5e282bde, 0x7e21bba4, 0x4e6c75fa, 0x5ac01217, 0x7f4304af, 0x1e7878ed, 0x1ada2196, 0x7ee1aba3, 0x93407f3c, 0x4f6c34eb, 0x6e3447a9, 0x7e7ae545, 0x5e0802bb, 0x6eeae63a, 0x7ee1da62, 0x5e280bb3, 0xf81d4009, 0x1e603b21, 0x5e281a14, 0x6eb0a99b, 0x1e266a25, 0x0d60cafe, 0x0e0b6194, 0x7a4ed2c5, 0x92b762ec, 0x4e6b5749, 0x3c16a6e5, 0x4ea0a92b, 0x0fa58b6a, 0x5f76148c, 0x6e30c95f, 0x1e6540fd, 0x5e28e40f, 0x0d403fd4, 0x7e30da36, 0x7fda9b51, 0x2ea04bde, 0x1e25c3d2, 0x1ee0434c, 0x5e21d8e7, 0x5ee1ba51, 0x5e61aba9, 0x4e2849fb, 0x5ee098ea, 0x4e60f63d, 0x0f280443, 0x5ee0da27, 0x2e78a6ce, 0x78054afc, 0x4e14286b, 0x4e218bd8, 0x2a3d2551, 0x3a04017a, 0x5f4317cd, 0x0e604a37, 0x9a834614, 0x0e2edf4d, 0x7a51a0a0, 0x5f8e9043, 0x6ea06bb2, 0xaa2857dd, 0x7a1903fc, 0x301ba9ba, 0x9ac929cd, 0x4e061ff0, 0x2e38fcfc, 0x0e2f614a, 0x7ee0d8e4, 0x6e73afda, 0x7f4156f7, 0x0e6078bf, 0x4ee1d9ed, 0x93403fbe, 0xce6f8640, 0x4e3855e3, 0x6f76fe23, 0x112466e8, 0x1e358a90, 0x7f45272c, 0x6ea19a9d, 0x8a696350, 0x1e3900f6, 0x5e61c866, 0x0e3fbfd0, 0x5ee09ad0, 0x0e651d27, 0x4dffc35e, 0x2e20c6ce, 0x0fbe118d, 0x1e656a15, 0xd1357365, 0x0e20a847, 0xce4a835c, 0x4e203905, 0x2e60090d, 0x7f4a27bb, 0x1e64c316, 0xce7d86a4, 0x7ebded2d, 0x6e70a97e, 0x4eb9a42b, 0x0e209bef, 0x6f151730, 0x0e7e30f7, 0x4e724509, 0xd503375f, 0xce58b6ae, 0x5e21a9b8, 0xcb2ca538, 0x5ac01131, 0x6ea19a24, 0xeb40c8b3, 0xc8df7d65, 0x78108341, 0x3218ab9b, 0x0f3da7dd, 0x2e003089, 0x4e21cab5, 0x8aa5c924, 0x1a94950c, 0x123e506f, 0x13117e37, 0x1ee6005b, 0x5ac00647, 0x5eec8cd5, 0x7ef0fb3d, 0x9223272a, 0x5ee0cb02, 0x6e66071d, 0x6ea1dbbf, 0x5e61c903, 0x5ac015ea, 0x93db6206, 0x7e62b5e3, 0x6ea0c87b, 0xdac0090e, 0x48df7d90, 0x6e206ba5, 0x9e2503c2, 0x6e25fc89, 0x4d60e2db, 0x1e3e22a0, 0x2eb81c19, 0x7856ea00, 0x5fbfb22d, 0x1e630244, 0x4e202a83, 0x1f50a722, 0x7f7b55d2, 0x0fae89b9, 0x4e781d73, 0xce738c3a, 0x4f15a591, 0x6e21c7e1, 0x586ff77e, 0x8a5d3592, 0x93401c67, 0x5e61cb86, 0xce6bc2c1, 0x6e393f10, 0x9bb70ec3, 0xdac0098c, 0x4da84b95, 0x7f494476, 0x9ace5c11, 0x7e61ca14, 0x4f7a60ef, 0x1ad32b39, 0x0ea3777f, 0x5e61da7f, 0x4f1404e2, 0x4e3244e2, 0x6e1b1ceb, 0x0dee5aac, 0x4e2f9dc4, 0x5ea1b8c3, 0x1e59f863, 0xd500403f, 0x4e3ae7d0, 0x4ef5c6ea, 0x08dffe3b, 0x6e36f4f6, 0x2e764f29, 0x0e726f23, 0x5f42375b, 0x7f71fc40, 0x6e618aad, 0x93403e5b, 0x0e205976, 0x0e7250c4, 0x6eb0abc9, 0x2e2049f0, 0x5f14754d, 0x7f6ce468, 0x6f950bbe, 0x6e31aa47, 0x4eb83396, 0x0dccc952, 0x2ea1ca90, 0xce69c701, 0xb0bed69e, 0x7c5dec39, 0x4e2868a2, 0x0e591b08, 0x5f34e6dd, 0x3a449184, 0x5e3ce6de, 0x4ea149b7, 0x4e7ad29b, 0xba198503, 0x1f683e8f, 0xfa52f2a7, 0x6e30dffc, 0x4e6c3d17, 0x2eae3248, 0xd503349f, 0x1e60002c, 0x0f180680, 0x9e240049, 0x6f75774e, 0xa90d8678, 0x9ad924c4, 0x7eb0f85b, 0x0e205aaf, 0x7ee08899, 0x5f4bffd8, 0x1b0ff5f3, 0x4ee11dcd, 0x2e218948, 0x0dcb2733, 0x4eac107c, 0x4ea04a53, 0x4e287b44, 0x0e60b82a, 0x5ee0ebbc, 0xce454ff1, 0x5e1761e7, 0x5e09202f, 0x0e0c0754, 0x1e72e6b9, 0x7e21da70, 0x0fbdb20c, 0x5efb8c84, 0xd500401f, 0x3a47526e, 0x1e680acf, 0x7f7375fc, 0xf80522da, 0x4ee60c02, 0x4d40c2e7, 0x6f89096b, 0x7ee1bb6e, 0x5e280b4a, 0x1e3120c8, 0x7eb2ef96, 0x4fd012dd, 0x0f3027ef, 0x4e2078a8, 0xd503201f, 0x2e2312d9, 0x6ebf1c6e, 0x5ee1f8df, 0x4e607a46, 0x6e30c877, 0x6c09d2d1, 0x4e61abd8, 0x0e35267e, 0x6ac17728, 0x0e861aa0, 0x6f63fe26, 0x6f157628, 0x6f30a5f9, 0x4d60cc0c, 0x4e21cb59, 0x2e68a3fb, 0x7efae601, 0x6ea0f82c, 0x9b25ec12, 0x1a1a0305, 0x0e043fe1, 0x6e73c0ed, 0x6ea1b8c0, 0x7e20380b, 0x0f0534e8, 0x1f56bc7d, 0xba0c0128, 0x1e672160, 0x6e7b259b, 0x7ee07b5d, 0x9a820443, 0x4e040581, 0x2f1d87e8, 0x1acd2f5b, 0x6e20794f, 0x2e6a3c93, 0xc8dffe13, 0xce5ab1c6, 0x6eea55f6, 0x4ea039b3, 0x0d602fec, 0x2e246e2f, 0x7857be39, 0xb80608fb, 0x1e67c017, 0x9bcf7f63, 0x0f92d857, 0x5e0812f7, 0x1e210172, 0x7e6128e9, 0x7ea94d41, 0x981179e1, 0x1effb018, 0x2e600828, 0x0eb9c6b2, 0x6ee1baae, 0x4ea0db28, 0x2ea1487b, 0x4ea6c7f0, 0x2e2374c7, 0x7e30d8dd, 0xb9991fa7, 0x4e791e3e, 0x889f7c4b, 0x0e6c753c, 0x1e740ad1, 0x1e244324, 0x1ef33010, 0x5ac01102, 0x9bd97fba, 0x6e290143, 0x1e2220d8, 0x4d8d5aee, 0x6f28570b, 0xfa4ab0c1, 0xdac00b14, 0x7ea1a90e, 0x2e3027d8, 0x6f25a733, 0x4e61a96e, 0x4e1a2fcb, 0x0e22fe0a, 0xc8df7cd0, 0x5e280a55, 0x4e012b20, 0x7e70dbf4, 0x520c5a4e, 0x6ea6c57f, 0x0e861af8, 0xd503233f, 0x889ffe3c, 0x5e274ea9, 0x4e21a89a, 0x0e170c02, 0x6efd4c0b, 0xd5033ebf, 0x6e61a92c, 0x2e205b72, 0x789fb828, 0x0e626e94, 0x2ea6724c, 0x9a10028b, 0x2c6c51fc, 0x5a9de6b9, 0x6e6881f3, 0x5ee0ea6b, 0x0faec36e, 0x0e955bca, 0x1acf206d, 0x7f6f571b, 0x4e286930, 0x12b41ceb, 0x1e770b7a, 0x0ea18ac2, 0x5e282aaf, 0xf2b7fa1e, 0x1ac34311, 0x13167d11, 0x4ea63412, 0x6e758038, 0x2f1d85d6, 0x0f275480, 0x0ead6c71, 0x6e204b69, 0x1e6303f4, 0x5e0031ef, 0x13001e40, 0x7a16006f, 0x6e6ae4c0, 0x0f0f242f, 0x6e674f50, 0x4e606b7a, 0x7e6ee684, 0x1e6b5957, 0x7ea1bbab, 0x7ea0b6cb, 0xce4da241, 0x0ea1b953, 0x0eb2af4b, 0x9ac309d0, 0x6e61d8bd, 0x5ea0d890, 0x5f47d1e7, 0xfa5981ca, 0x1e7f7959, 0x6ef24dd8, 0x0e0a41d1, 0x5ee0e898, 0x4e6038e2, 0x13097d65, 0x6f839088, 0x9e290265, 0x0e208824, 0x2e65af79, 0x6f36a561, 0x9ad3204b, 0x0e21482e, 0x1e24431d, 0xd50330bf, 0x0df641aa, 0x6e602a83, 0xce30505f, 0x5e025238, 0xd503201f, 0x4e608880, 0x4de9c38d, 0x5e0f5348, 0x6eb48ca9, 0x50fda31b, 0x2e251eec, 0x7842ba50, 0xd8a1cd86, 0x2ea09862, 0x0ea09983, 0x2ea333b0, 0x0ea6032c, 0x4f94801b, 0x7e3ee57d, 0x38135e4f, 0xd8fdd9dd, 0x5ee0fcde, 0x9e64033d, 0x6e37f547, 0x6e3dd7ef, 0x13003f3d, 0x0e602f9f, 0x4e7ad014, 0x9b3b6857, 0x5ea0cb67, 0x0eb31c9f, 0x4e7c5372, 0x5e61b8c0, 0x0ea19b23, 0x0ee6e1df, 0x6e63a626, 0x2f139405, 0x7eb0f96d, 0x9e588c63, 0x2e714c3a, 0x6e8c941e, 0x0f61b331, 0x6f01f625, 0x4e78d4ea, 0x6f403709, 0x1a0300da, 0xda0102c8, 0x7e61d9fd, 0xb89469bb, 0x0c838780, 0x2e60a590, 0x4dfd29e1, 0x4e150f2e, 0xce2810bc, 0x5f541591, 0x9ee60259, 0x2eb40e56, 0x5e014027, 0x2ef71faf, 0x4e2d452f, 0x5ee0a813, 0x4eb03301, 0x38443acf, 0x6eabd502, 0x0e2ee71e, 0x5a960364, 0xce7ec596, 0x7efbed09, 0x4ef42ea2, 0x0eb30ea5, 0x5ee0d9f8, 0x6f513552, 0xf89eb3fa, 0x7ea2eca6, 0x9b00cc19, 0xf897409e, 0x1e73485f, 0x381afa77, 0x0f169f3b, 0x5ee1aa70, 0x5e1803ee, 0x0dbf5a4c, 0xce78c7a6, 0x9b0b260c, 0x2ef8fa19, 0x6e70aa4b, 0xce45b805, 0x2ea08e86, 0x4ee0bafd, 0x2ea09a1f, 0x4e218900, 0x6e744f13, 0xce518653, 0xf81b7a68, 0xce45ac5e, 0x7e62e416, 0x1a1b02b6, 0x7e21db48, 0x381daaaf, 0x6b2c0987, 0x0e2ec651, 0x4eae8502, 0x9bde7ca0, 0x6f47201f, 0x7e61a8a3, 0x6e60d5db, 0x4e2879de, 0xf81d194e, 0x4f1b8d05, 0x4d0048b2, 0x6e203be9, 0x4e3e7eb1, 0x0e260ef8, 0x2e688518, 0x7e3fec46, 0xdac00843, 0xf85c8917, 0x2e212a0f, 0x0e8196da, 0xd503359f, 0xce4c81f2, 0x6ee19992, 0x6e21ca79, 0x4d40c1d2, 0x4f5816ef, 0x4e34c3ea, 0x4df7c283, 0x7ef7eeb6, 0x18e276ce, 0xab0d21c0, 0xd5032f7f, 0x4ea00dbf, 0x5ac01251, 0xd0121955, 0x7f1495e4, 0x7ef0fa11, 0x5e24dd9c, 0x9add25b5, 0x0eb2bdef, 0x9e1977c7, 0x6f4b26bd, 0x0e200a9c, 0x9b4f7c00, 0x0ea0392e, 0x7e212a2c, 0x0b248b90, 0x1acc27a1, 0x2e701c90, 0x5ee1b870, 0x5e280aba, 0x5ea0780e, 0x1e264246, 0x4e052d04, 0x0e731dc4, 0xce461997, 0x9a9e9413, 0x3d462048, 0x5ea1fac5, 0x2ea0c8c4, 0x9a030280, 0x2ebda4b8, 0x5eef8614, 0x6eadc4e0, 0xbd035a8f, 0x4e606b84, 0x4eb1aba1, 0x4e286928, 0x4e2858cc, 0x9add0ce9, 0x4e070d65, 0x5fd399d5, 0x0f03fde7, 0x6ee90c74, 0x4ef8e31e, 0x381d986a, 0x5ea0ebf4, 0x5ea0d87e, 0x2e76ac9e, 0x6eb36cd4, 0x2e6e1c4c, 0x2e2feebc, 0x1ace4b03, 0x5ee0db12, 0x5ea0e9b1, 0x2e1c32d5, 0x5fa49a09, 0x0e258737, 0x7e21ca8e, 0xce4f9988, 0x5f7f56a6, 0x0e739766, 0x4e28586c, 0x6e619908, 0xd500401f, 0xf88b9252, 0x6e251c8e, 0x9e20015b, 0x7f1486b9, 0x717c339b, 0x1f31ff70, 0x4ea0eb62, 0x9acb0926, 0x489f7d85, 0x4e209b54, 0x2e84cf03, 0x2e65946c, 0x0e7d80cd, 0xc8dffecc, 0xce668bd8, 0x6e2188af, 0xeb4ada34, 0x2b25ec33, 0x0d40e6e7, 0x4eb2c757, 0x4ec82ad0, 0x7e21cb0a, 0x0e21a847, 0x4e0b1ec0, 0x381e6ac0, 0x6e61c8f5, 0x0f10071c, 0x2ee21daa, 0x5e61ab31, 0x6e218892, 0x2e7e7cb5, 0x6f2826aa, 0x7f6b54df, 0x4eaa2620, 0xdac00034, 0x4f6477be, 0x7e6148ea, 0x4eef1f57, 0x78459aeb, 0x2ebc3f10, 0x2e35f4eb, 0x4fbf19ce, 0xd8d0e58e, 0x2e21bbc7, 0x6ee0cab6, 0x9bc57e3f, 0x2f854037, 0x4e92181c, 0x6e6d1f89, 0x0f305545, 0x4ee19a57, 0x0e887bdf, 0x5e1a4185, 0x7ef0c821, 0x2eb6607c, 0x2ea0d9b8, 0x9e0380f4, 0x2ebf1c83, 0x1e62597d, 0x7f6e2548, 0x5ac00205, 0x4e616adb, 0xce638b8c, 0x5e1653cf, 0x2e6069be, 0x0e2ac641, 0x1e33c76f, 0xce44956d, 0x9bb90d31, 0x1e24c20a, 0x7ee038c1, 0x93407e5e, 0x4e280127, 0xc8df7f7d, 0xba42f263, 0x1e6f199c, 0x6e212889, 0x6e92f60e, 0x6ebdc499, 0x8b9acbf8, 0x4d40c581, 0x3a020250, 0x6e6a6716, 0x9248403b, 0x9081ffea, 0x4e603856, 0x9ad1242b, 0x6f270579, 0x1a070349, 0xcec08133, 0xd503305f, 0x5a1a00ca, 0x2e60b8a2, 0x0e5f28fd, 0x0e31a3da, 0x7e61cbc1, 0xd503399f, 0x5f5e54aa, 0x0eb8bdea, 0x4eba8f10, 0x4e2a2e60, 0x2f3da7d6, 0x1e58e297, 0x6e71aa3e, 0x6b86701a, 0xce4fa5e6, 0x4ee7c463, 0x8a79307f, 0x0ebea541, 0x2e218af4, 0x4e774f8a, 0xb9b95dc5, 0x6e61abd5, 0x4dd1e814, 0x4da72098, 0x98307582, 0x3a512101, 0x7ef95497, 0x1ace5535, 0x5a0c0349, 0x4e28581b, 0x6ebf1c02, 0x5ea1da23, 0x1e274314, 0x5e25dd29, 0x6e75f594, 0x6eaf6ed5, 0x4e214abe, 0x4e064172, 0x2e21c8f4, 0xf84c5b08, 0x1e244312, 0x14000000};
for (size_t i = 0; i < code32.size(); ++i)
- env.MemoryWrite32(100 + i, code32[i]);
+ env.MemoryWrite(100 + i, code32[i], sizeof(u32));
env.ignore_invalid_insn = true;
jit.SetRegister(0, 0x866524401a1d4e47);
diff --git a/src/dynarmic/tests/A64/fibonacci.cpp b/src/dynarmic/tests/A64/fibonacci.cpp
index f52a04e217..8816a89b1c 100644
--- a/src/dynarmic/tests/A64/fibonacci.cpp
+++ b/src/dynarmic/tests/A64/fibonacci.cpp
@@ -25,48 +25,55 @@ public:
u64 ticks_left = 0;
::Common::unordered_map<u64, u8> memory{};
- u8 MemoryRead8(u64 vaddr) override {
- return memory[vaddr];
- }
-
- u16 MemoryRead16(u64 vaddr) override {
- return u16(MemoryRead8(vaddr)) | u16(MemoryRead8(vaddr + 1)) << 8;
- }
-
- u32 MemoryRead32(u64 vaddr) override {
- return u32(MemoryRead16(vaddr)) | u32(MemoryRead16(vaddr + 2)) << 16;
- }
-
- u64 MemoryRead64(u64 vaddr) override {
- return u64(MemoryRead32(vaddr)) | u64(MemoryRead32(vaddr + 4)) << 32;
+ u64 MemoryRead(u64 vaddr, size_t size) override {
+ switch (size) {
+ case sizeof(u64):
+ return MemoryRead(vaddr, sizeof(u32))
+ | MemoryRead(vaddr + sizeof(u32), sizeof(u32)) << 32;
+ case sizeof(u32):
+ return MemoryRead(vaddr, sizeof(u16))
+ | MemoryRead(vaddr + sizeof(u16), sizeof(u16)) << 16;
+ case sizeof(u16):
+ return MemoryRead(vaddr, sizeof(u8))
+ | MemoryRead(vaddr + sizeof(u8), sizeof(u8)) << 8;
+ case sizeof(u8):
+ return memory[vaddr];
+ default:
+ std::abort();
+ }
}
std::array<u64, 2> MemoryRead128(u64 vaddr) override {
- return {MemoryRead64(vaddr), MemoryRead64(vaddr + 8)};
- }
-
- void MemoryWrite8(u64 vaddr, u8 value) override {
- memory[vaddr] = value;
+ return {
+ MemoryRead(vaddr, sizeof(u64)),
+ MemoryRead(vaddr + sizeof(u64), sizeof(u64))
+ };
+ }
+
+ void MemoryWrite(Dynarmic::A64::VAddr vaddr, u64 value, size_t size) override {
+ switch (size) {
+ case sizeof(u64):
+ MemoryWrite(vaddr, u32(value), sizeof(u32));
+ MemoryWrite(vaddr + 4, u32(value >> 32), sizeof(u32));
+ break;
+ case sizeof(u32):
+ MemoryWrite(vaddr, u16(value), sizeof(u16));
+ MemoryWrite(vaddr + 2, u16(value >> 16), sizeof(u16));
+ break;
+ case sizeof(u16):
+ MemoryWrite(vaddr, u8(value), sizeof(u8));
+ MemoryWrite(vaddr + 1, u8(value >> 8), sizeof(u8));
+ break;
+ case sizeof(u8):
+ memory[vaddr] = value;
+ break;
+ default:
+ std::abort();
+ }
}
-
- void MemoryWrite16(u64 vaddr, u16 value) override {
- MemoryWrite8(vaddr, u8(value));
- MemoryWrite8(vaddr + 1, u8(value >> 8));
- }
-
- void MemoryWrite32(u64 vaddr, u32 value) override {
- MemoryWrite16(vaddr, u16(value));
- MemoryWrite16(vaddr + 2, u16(value >> 16));
- }
-
- void MemoryWrite64(u64 vaddr, u64 value) override {
- MemoryWrite32(vaddr, u32(value));
- MemoryWrite32(vaddr + 4, u32(value >> 32));
- }
-
void MemoryWrite128(u64 vaddr, std::array<u64, 2> value) override {
- MemoryWrite64(vaddr, value[0]);
- MemoryWrite64(vaddr + 8, value[1]);
+ MemoryWrite(vaddr, value[0], sizeof(u64));
+ MemoryWrite(vaddr + 8, value[1], sizeof(u64));
}
void CallSVC(u32) override {
@@ -135,9 +142,9 @@ TEST_CASE("A64: fibonacci", "[a64]") {
code.RET();
for (size_t i = 0; i < 1024; i++) {
- env.MemoryWrite32(i * 4, instructions[i]);
+ env.MemoryWrite(i * 4, instructions[i], sizeof(u32));
}
- env.MemoryWrite32(8888, 0xd4200000);
+ env.MemoryWrite(8888, 0xd4200000, sizeof(u32));
cpu.SetRegister(30, 8888);
cpu.SetRegister(0, 10);
diff --git a/src/dynarmic/tests/A64/testenv.h b/src/dynarmic/tests/A64/testenv.h
index f4ddc63a6b..11aa6389f8 100644
--- a/src/dynarmic/tests/A64/testenv.h
+++ b/src/dynarmic/tests/A64/testenv.h
@@ -12,6 +12,7 @@
#include "common/assert.h"
#include "common/common_types.h"
#include "dynarmic/interface/A64/a64.h"
+#include "dynarmic/interface/A64/config.h"
using Vector = Dynarmic::A64::Vector;
@@ -34,64 +35,79 @@ public:
return code_mem[index];
}
- std::uint8_t MemoryRead8(u64 vaddr) override {
- if (IsInCodeMem(vaddr)) {
- return reinterpret_cast<u8*>(code_mem.data())[vaddr - code_mem_start_address];
+ u64 MemoryRead(u64 vaddr, size_t size) override {
+ switch (size) {
+ case sizeof(u64):
+ return MemoryRead(vaddr, sizeof(u32))
+ | MemoryRead(vaddr + sizeof(u32), sizeof(u32)) << 32;
+ case sizeof(u32):
+ return MemoryRead(vaddr, sizeof(u16))
+ | MemoryRead(vaddr + sizeof(u16), sizeof(u16)) << 16;
+ case sizeof(u16):
+ return MemoryRead(vaddr, sizeof(u8))
+ | MemoryRead(vaddr + sizeof(u8), sizeof(u8)) << 8;
+ case sizeof(u8): {
+ if (IsInCodeMem(vaddr))
+ return reinterpret_cast<u8*>(code_mem.data())[vaddr - code_mem_start_address];
+ if (auto const it = modified_memory.find(vaddr); it != modified_memory.end())
+ return it->second;
+ return u8(vaddr);
+ }
+ default:
+ std::abort();
}
- if (auto const it = modified_memory.find(vaddr); it != modified_memory.end())
- return it->second;
- return u8(vaddr);
- }
- std::uint16_t MemoryRead16(u64 vaddr) override {
- return u16(MemoryRead8(vaddr)) | u16(MemoryRead8(vaddr + 1)) << 8;
- }
- std::uint32_t MemoryRead32(u64 vaddr) override {
- return u32(MemoryRead16(vaddr)) | u32(MemoryRead16(vaddr + 2)) << 16;
- }
- std::uint64_t MemoryRead64(u64 vaddr) override {
- return u64(MemoryRead32(vaddr)) | u64(MemoryRead32(vaddr + 4)) << 32;
- }
- Vector MemoryRead128(u64 vaddr) override {
- return {MemoryRead64(vaddr), MemoryRead64(vaddr + 8)};
}
- void MemoryWrite8(u64 vaddr, std::uint8_t value) override {
- if (IsInCodeMem(vaddr)) {
- code_mem_modified_by_guest = true;
+ Vector MemoryRead128(u64 vaddr) override {
+ return {
+ MemoryRead(vaddr, sizeof(u64)),
+ MemoryRead(vaddr + 8, sizeof(u64))
+ };
+ }
+
+ void MemoryWrite(Dynarmic::A64::VAddr vaddr, u64 value, size_t size) override {
+ switch (size) {
+ case sizeof(u64):
+ MemoryWrite(vaddr, u32(value), sizeof(u32));
+ MemoryWrite(vaddr + 4, u32(value >> 32), sizeof(u32));
+ break;
+ case sizeof(u32):
+ MemoryWrite(vaddr, u16(value), sizeof(u16));
+ MemoryWrite(vaddr + 2, u16(value >> 16), sizeof(u16));
+ break;
+ case sizeof(u16):
+ MemoryWrite(vaddr, u8(value), sizeof(u8));
+ MemoryWrite(vaddr + 1, u8(value >> 8), sizeof(u8));
+ break;
+ case sizeof(u8):
+ if (IsInCodeMem(vaddr)) {
+ code_mem_modified_by_guest = true;
+ }
+ modified_memory[vaddr] = value;
+ break;
+ default:
+ std::abort();
}
- modified_memory[vaddr] = value;
- }
- void MemoryWrite16(u64 vaddr, std::uint16_t value) override {
- MemoryWrite8(vaddr, u8(value));
- MemoryWrite8(vaddr + 1, u8(value >> 8));
- }
- void MemoryWrite32(u64 vaddr, std::uint32_t value) override {
- MemoryWrite16(vaddr, u16(value));
- MemoryWrite16(vaddr + 2, u16(value >> 16));
- }
- void MemoryWrite64(u64 vaddr, std::uint64_t value) override {
- MemoryWrite32(vaddr, u32(value));
- MemoryWrite32(vaddr + 4, u32(value >> 32));
}
void MemoryWrite128(u64 vaddr, Vector value) override {
- MemoryWrite64(vaddr, value[0]);
- MemoryWrite64(vaddr + 8, value[1]);
+ MemoryWrite(vaddr, value[0], sizeof(u64));
+ MemoryWrite(vaddr + 8, value[1], sizeof(u64));
}
bool MemoryWriteExclusive8(u64 vaddr, std::uint8_t value, [[maybe_unused]] std::uint8_t expected) override {
- MemoryWrite8(vaddr, value);
+ MemoryWrite(vaddr, value, sizeof(u8));
return true;
}
bool MemoryWriteExclusive16(u64 vaddr, std::uint16_t value, [[maybe_unused]] std::uint16_t expected) override {
- MemoryWrite16(vaddr, value);
+ MemoryWrite(vaddr, value, sizeof(u16));
return true;
}
bool MemoryWriteExclusive32(u64 vaddr, std::uint32_t value, [[maybe_unused]] std::uint32_t expected) override {
- MemoryWrite32(vaddr, value);
+ MemoryWrite(vaddr, value, sizeof(u32));
return true;
}
bool MemoryWriteExclusive64(u64 vaddr, std::uint64_t value, [[maybe_unused]] std::uint64_t expected) override {
- MemoryWrite64(vaddr, value);
+ MemoryWrite(vaddr, value, sizeof(u64));
return true;
}
bool MemoryWriteExclusive128(u64 vaddr, Vector value, [[maybe_unused]] Vector expected) override {
@@ -145,52 +161,46 @@ public:
return read<std::uint32_t>(vaddr);
}
- std::uint8_t MemoryRead8(u64 vaddr) override {
- return read<std::uint8_t>(vaddr);
- }
- std::uint16_t MemoryRead16(u64 vaddr) override {
- return read<std::uint16_t>(vaddr);
- }
- std::uint32_t MemoryRead32(u64 vaddr) override {
- return read<std::uint32_t>(vaddr);
- }
- std::uint64_t MemoryRead64(u64 vaddr) override {
- return read<std::uint64_t>(vaddr);
+ u64 MemoryRead(u64 vaddr, size_t size) override {
+ switch (size) {
+ case sizeof(u64): return read<u64>(vaddr);
+ case sizeof(u32): return read<u32>(vaddr);
+ case sizeof(u16): return read<u16>(vaddr);
+ case sizeof(u8): return read<u8>(vaddr);
+ default: std::abort();
+ }
}
Vector MemoryRead128(u64 vaddr) override {
return read<Vector>(vaddr);
}
- void MemoryWrite8(u64 vaddr, std::uint8_t value) override {
- write(vaddr, value);
- }
- void MemoryWrite16(u64 vaddr, std::uint16_t value) override {
- write(vaddr, value);
- }
- void MemoryWrite32(u64 vaddr, std::uint32_t value) override {
- write(vaddr, value);
- }
- void MemoryWrite64(u64 vaddr, std::uint64_t value) override {
- write(vaddr, value);
+ void MemoryWrite(u64 vaddr, std::uint64_t value, size_t size) override {
+ switch (size) {
+ case sizeof(u64): return write<u64>(vaddr, u64(value));
+ case sizeof(u32): return write<u32>(vaddr, u32(value));
+ case sizeof(u16): return write<u16>(vaddr, u16(value));
+ case sizeof(u8): return write<u8>(vaddr, u8(value));
+ default: std::abort();
+ }
}
void MemoryWrite128(u64 vaddr, Vector value) override {
write(vaddr, value);
}
bool MemoryWriteExclusive8(u64 vaddr, std::uint8_t value, [[maybe_unused]] std::uint8_t expected) override {
- MemoryWrite8(vaddr, value);
+ MemoryWrite(vaddr, value, sizeof(u8));
return true;
}
bool MemoryWriteExclusive16(u64 vaddr, std::uint16_t value, [[maybe_unused]] std::uint16_t expected) override {
- MemoryWrite16(vaddr, value);
+ MemoryWrite(vaddr, value, sizeof(u16));
return true;
}
bool MemoryWriteExclusive32(u64 vaddr, std::uint32_t value, [[maybe_unused]] std::uint32_t expected) override {
- MemoryWrite32(vaddr, value);
+ MemoryWrite(vaddr, value, sizeof(u32));
return true;
}
bool MemoryWriteExclusive64(u64 vaddr, std::uint64_t value, [[maybe_unused]] std::uint64_t expected) override {
- MemoryWrite64(vaddr, value);
+ MemoryWrite(vaddr, value, sizeof(u64));
return true;
}
bool MemoryWriteExclusive128(u64 vaddr, Vector value, [[maybe_unused]] Vector expected) override {
diff --git a/src/dynarmic/tests/print_info.cpp b/src/dynarmic/tests/print_info.cpp
index 5b1c5cfe63..87f4438ad4 100644
--- a/src/dynarmic/tests/print_info.cpp
+++ b/src/dynarmic/tests/print_info.cpp
@@ -18,6 +18,7 @@
#include <fmt/format.h>
#include <fmt/ostream.h>
#include <fmt/ranges.h>
+#include "dynarmic/frontend/A32/translate/translate_callbacks.h"
#include "dynarmic/mcl/bit.hpp"
#include "common/common_types.h"
@@ -118,36 +119,47 @@ public:
u64 ticks_left = 0;
std::map<u32, u8> memory;
- std::uint8_t MemoryRead8(u32 vaddr) override {
- if (auto iter = memory.find(vaddr); iter != memory.end()) {
- return iter->second;
+ u64 MemoryRead(Dynarmic::A32::VAddr vaddr, size_t size) override {
+ switch (size) {
+ case sizeof(u64):
+ return MemoryRead(vaddr, sizeof(u32))
+ | MemoryRead(vaddr + sizeof(u32), sizeof(u32)) << 32;
+ case sizeof(u32):
+ return MemoryRead(vaddr, sizeof(u16))
+ | MemoryRead(vaddr + sizeof(u16), sizeof(u16)) << 16;
+ case sizeof(u16):
+ return MemoryRead(vaddr, sizeof(u8))
+ | MemoryRead(vaddr + sizeof(u8), sizeof(u8)) << 8;
+ case sizeof(u8): {
+ if (auto const it = memory.find(vaddr); it != memory.end())
+ return it->second;
+ return 0;
+ }
+ default:
+ std::abort();
}
- return 0;
- }
- std::uint16_t MemoryRead16(u32 vaddr) override {
- return u16(MemoryRead8(vaddr)) | u16(MemoryRead8(vaddr + 1)) << 8;
- }
- std::uint32_t MemoryRead32(u32 vaddr) override {
- return u32(MemoryRead16(vaddr)) | u32(MemoryRead16(vaddr + 2)) << 16;
- }
- std::uint64_t MemoryRead64(u32 vaddr) override {
- return u64(MemoryRead32(vaddr)) | u64(MemoryRead32(vaddr + 4)) << 32;
}
- void MemoryWrite8(u32 vaddr, std::uint8_t value) override {
- memory[vaddr] = value;
- }
- void MemoryWrite16(u32 vaddr, std::uint16_t value) override {
- MemoryWrite8(vaddr, static_cast<u8>(value));
- MemoryWrite8(vaddr + 1, static_cast<u8>(value >> 8));
- }
- void MemoryWrite32(u32 vaddr, std::uint32_t value) override {
- MemoryWrite16(vaddr, static_cast<u16>(value));
- MemoryWrite16(vaddr + 2, static_cast<u16>(value >> 16));
- }
- void MemoryWrite64(u32 vaddr, std::uint64_t value) override {
- MemoryWrite32(vaddr, static_cast<u32>(value));
- MemoryWrite32(vaddr + 4, static_cast<u32>(value >> 32));
+ void MemoryWrite(Dynarmic::A32::VAddr vaddr, u64 value, size_t size) override {
+ switch (size) {
+ case sizeof(u64):
+ MemoryWrite(vaddr, u32(value), sizeof(u32));
+ MemoryWrite(vaddr + 4, u32(value >> 32), sizeof(u32));
+ break;
+ case sizeof(u32):
+ MemoryWrite(vaddr, u16(value), sizeof(u16));
+ MemoryWrite(vaddr + 2, u16(value >> 16), sizeof(u16));
+ break;
+ case sizeof(u16):
+ MemoryWrite(vaddr, u8(value), sizeof(u8));
+ MemoryWrite(vaddr + 1, u8(value >> 8), sizeof(u8));
+ break;
+ case sizeof(u8):
+ memory[vaddr] = value;
+ break;
+ default:
+ std::abort();
+ }
}
void CallSVC(std::uint32_t swi) override {
@@ -231,7 +243,7 @@ void ExecuteA32Instruction(u32 instruction) {
if (const auto address = get_value()) {
fmt::print("value: ");
if (const auto value = get_value()) {
- env.MemoryWrite32(*address, *value);
+ env.MemoryWrite(*address, *value, sizeof(u32));
fmt::print("> mem[{:#08x}] = {:#08x}\n", *address, *value);
}
}
@@ -247,8 +259,8 @@ void ExecuteA32Instruction(u32 instruction) {
cpu.SetFpscr(fpscr);
const u32 initial_pc = regs[15];
- env.MemoryWrite32(initial_pc + 0, instruction);
- env.MemoryWrite32(initial_pc + 4, 0xEAFFFFFE); // B +0
+ env.MemoryWrite(initial_pc + 0, instruction, sizeof(u32));
+ env.MemoryWrite(initial_pc + 4, 0xEAFFFFFE, sizeof(u32)); // B +0
cpu.Run();
fmt::print("{}", fmt::join(cpu.Disassemble(), "\n"));
diff --git a/src/dynarmic/tests/unicorn_emu/a32_unicorn.cpp b/src/dynarmic/tests/unicorn_emu/a32_unicorn.cpp
index c0e1e13d9b..1fe09cdef0 100644
--- a/src/dynarmic/tests/unicorn_emu/a32_unicorn.cpp
+++ b/src/dynarmic/tests/unicorn_emu/a32_unicorn.cpp
@@ -290,7 +290,7 @@ bool A32Unicorn<TestEnvironment>::UnmappedMemoryHook(uc_engine* uc, uc_mem_type
auto page = std::make_unique<Page>();
page->address = base_address;
for (size_t i = 0; i < page->data.size(); ++i)
- page->data[i] = this_->testenv.MemoryRead8(static_cast<u32>(base_address + i));
+ page->data[i] = u8(this_->testenv.MemoryRead(u32(base_address + i), sizeof(u8)));
uc_err err = uc_mem_map_ptr(uc, base_address, page->data.size(), permissions, page->data.data());
if (err == UC_ERR_MAP)
@@ -321,24 +321,7 @@ bool A32Unicorn<TestEnvironment>::UnmappedMemoryHook(uc_engine* uc, uc_mem_type
template<class TestEnvironment>
bool A32Unicorn<TestEnvironment>::MemoryWriteHook(uc_engine* /*uc*/, uc_mem_type /*type*/, u32 start_address, int size, u64 value, void* user_data) {
auto* this_ = static_cast<A32Unicorn*>(user_data);
-
- switch (size) {
- case 1:
- this_->testenv.MemoryWrite8(start_address, static_cast<u8>(value));
- break;
- case 2:
- this_->testenv.MemoryWrite16(start_address, static_cast<u16>(value));
- break;
- case 4:
- this_->testenv.MemoryWrite32(start_address, static_cast<u32>(value));
- break;
- case 8:
- this_->testenv.MemoryWrite64(start_address, value);
- break;
- default:
- UNREACHABLE();
- }
-
+ this_->testenv.MemoryWrite(start_address, value, size);
return true;
}
diff --git a/src/dynarmic/tests/unicorn_emu/a64_unicorn.cpp b/src/dynarmic/tests/unicorn_emu/a64_unicorn.cpp
index c9a194d50a..4afb6a6859 100644
--- a/src/dynarmic/tests/unicorn_emu/a64_unicorn.cpp
+++ b/src/dynarmic/tests/unicorn_emu/a64_unicorn.cpp
@@ -197,7 +197,7 @@ bool A64Unicorn::UnmappedMemoryHook(uc_engine* uc, uc_mem_type /*type*/, u64 sta
auto page = std::make_unique<Page>();
page->address = base_address;
for (size_t i = 0; i < page->data.size(); ++i)
- page->data[i] = this_->testenv.MemoryRead8(base_address + i);
+ page->data[i] = u8(this_->testenv.MemoryRead(base_address + i, sizeof(u8)));
uc_err err = uc_mem_map_ptr(uc, base_address, page->data.size(), permissions, page->data.data());
if (err == UC_ERR_MAP)
@@ -227,23 +227,6 @@ bool A64Unicorn::UnmappedMemoryHook(uc_engine* uc, uc_mem_type /*type*/, u64 sta
bool A64Unicorn::MemoryWriteHook(uc_engine* /*uc*/, uc_mem_type /*type*/, u64 start_address, int size, u64 value, void* user_data) {
auto* this_ = static_cast<A64Unicorn*>(user_data);
-
- switch (size) {
- case 1:
- this_->testenv.MemoryWrite8(start_address, static_cast<u8>(value));
- break;
- case 2:
- this_->testenv.MemoryWrite16(start_address, static_cast<u16>(value));
- break;
- case 4:
- this_->testenv.MemoryWrite32(start_address, static_cast<u32>(value));
- break;
- case 8:
- this_->testenv.MemoryWrite64(start_address, value);
- break;
- default:
- UNREACHABLE();
- }
-
+ this_->testenv.MemoryWrite(start_address, value, size);
return true;
}