From 9ae385c77474d75843510f56d803fbd70d6a8d66 Mon Sep 17 00:00:00 2001 From: Luke Kenneth Casson Leighton Date: Wed, 29 Nov 2023 19:29:12 +0000 Subject: [PATCH] bug #672: pospopcount working with large arrays pospopcount is supposed to be able to handle long arrays of data, but it turns out that sv.lbzu/pi/dw=8 was calculating an EA in 8-bit, meaning that it wrapped around to a zero memory address. now this is resolved the code which has been made shorter actually works --- src/openpower/decoder/isa/test_caller_svp64_pospopcount.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/openpower/decoder/isa/test_caller_svp64_pospopcount.py b/src/openpower/decoder/isa/test_caller_svp64_pospopcount.py index 8c73ce71..06b1d764 100644 --- a/src/openpower/decoder/isa/test_caller_svp64_pospopcount.py +++ b/src/openpower/decoder/isa/test_caller_svp64_pospopcount.py @@ -60,21 +60,24 @@ class PosPopCountTestCase(FHDLTestCase): #tst_array = [254] * 10 #tst_array = [1,2,3,4,5,6,7,8,9,10,11,12,13] #tst_array = [1,2,3,4,5,6,7,8,9,10,11,12,13] + #tst_array = [1,2,3,4,5,6,7,8,9] tst_array = list(range(240)) initial_regs = [0] * 64 initial_regs[3] = len(tst_array) - initial_regs[4] = 16 # load address + initial_regs[4] = 256-8 # load address # some memory with identifying garbage in it initial_mem = {16: 0xf0f1_f2f3_f4f5_f6f7, 24: 0x4041_4243_4445_4647, 40: 0x8081_8283_8485_8687, 48: 0x9091_9293_9495_9697, + 248: 0xffff_aaaa_cccc_eeee, + 256: 0xa0a1_a2a3_a4a5_a6a7, } # overwrite the garbage with the test data for i, c in enumerate(tst_array): - write_byte(initial_mem, 16+i, c) + write_byte(initial_mem, initial_regs[4]+i, c) for i, c in enumerate(tst_array): print ("array", i, bin(c), c) -- 2.30.2