From c5cbe437327e951cc6e520cc430b87a1754cf6fa Mon Sep 17 00:00:00 2001 From: Kip Macsai-Goren Date: Sun, 18 Sep 2022 00:09:52 +0000 Subject: [PATCH] Fixed typos in existing endianness test --- .../references/WALLY-endianness-01.reference_output | 2 +- .../rv64i_m/privilege/src/WALLY-endianness-01.S | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/wally-riscv-arch-test/riscv-test-suite/rv64i_m/privilege/references/WALLY-endianness-01.reference_output b/tests/wally-riscv-arch-test/riscv-test-suite/rv64i_m/privilege/references/WALLY-endianness-01.reference_output index e1984a7cb..349a2bca3 100644 --- a/tests/wally-riscv-arch-test/riscv-test-suite/rv64i_m/privilege/references/WALLY-endianness-01.reference_output +++ b/tests/wally-riscv-arch-test/riscv-test-suite/rv64i_m/privilege/references/WALLY-endianness-01.reference_output @@ -1,4 +1,4 @@ -eeff1122 # Test *** Number: M mode little endian load/store double of 0xaabbccddeeff1122 +eeff1122 # Test 5.3.2.4: M mode little endian load/store double of 0xaabbccddeeff1122 aabbccdd # NOTE: since we're doing a store that matches the width of the load, we cut out all the sign extension eeff1122 # M mode little endian load/store word of 0xaabbccddeeff1122 deadbeef # NOTE: the memory was already filled with deadbeef's so subword overwrite some, but not all of them. this is why the values are filled with deadbeefs, rather than 00's or ff's diff --git a/tests/wally-riscv-arch-test/riscv-test-suite/rv64i_m/privilege/src/WALLY-endianness-01.S b/tests/wally-riscv-arch-test/riscv-test-suite/rv64i_m/privilege/src/WALLY-endianness-01.S index f3fe822b8..38bc6fd18 100644 --- a/tests/wally-riscv-arch-test/riscv-test-suite/rv64i_m/privilege/src/WALLY-endianness-01.S +++ b/tests/wally-riscv-arch-test/riscv-test-suite/rv64i_m/privilege/src/WALLY-endianness-01.S @@ -30,7 +30,7 @@ INIT_TESTS TRAP_HANDLER m -// Test ***Number: testing that accesses to sub-word memory acceses not on a word boundary go +// Test 5.3.2.4: testing that accesses to sub-word memory acceses not on a word boundary go // correctly with the relevant status bit indicating endianness for each mode. // do this by going to each mode with and testing loads with the big endian bit on and off work correctly @@ -70,7 +70,7 @@ csrs mstatus, x28 // turn on big endianness for M mode li x28, 0xAABBCCDDEEFF1122 li x29, 0x8000F000 -sd x28, 0(x29) // value stored in memory as 0x2211FFEECCDDBBAA +sd x28, 0(x29) // value stored in memory as 0x2211FFEEDDCCBBAA ld x30, 0(x29) // test load double, should read out 0xAABBCCDDEEFF1122 sd x30, 0(t1) // test store double, should save 0x2211FFEEDDCCBBAA @@ -134,7 +134,7 @@ GOTO_S_MODE li x28, 0xAABBCCDDEEFF1122 li x29, 0x8000F000 -sd x28, 0(x29) // value stored in memory as 0x2211FFEECCDDBBAA +sd x28, 0(x29) // value stored in memory as 0x2211FFEEDDCCBBAA ld x30, 0(x29) // test load double, should read out 0xAABBCCDDEEFF1122 sd x30, 0(t1) // test store double, should save 0x2211FFEEDDCCBBAA @@ -195,7 +195,7 @@ GOTO_U_MODE li x28, 0xAABBCCDDEEFF1122 li x29, 0x8000F000 -sd x28, 0(x29) // value stored in memory as 0x2211FFEECCDDBBAA +sd x28, 0(x29) // value stored in memory as 0x2211FFEEDDCCBBAA ld x30, 0(x29) // test load double, should read out 0xAABBCCDDEEFF1122 sd x30, 0(t1) // test store double, should save 0x2211FFEEDDCCBBAA