From a0c38e64cd71d724e38b588e76a100885bca2662 Mon Sep 17 00:00:00 2001 From: Luke Kenneth Casson Leighton Date: Mon, 19 Jul 2021 14:46:25 +0100 Subject: [PATCH] no need for len(j) > 1 test, half of 1 is zero which stops swap anyway --- src/openpower/decoder/isa/fastdctlee.py | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/src/openpower/decoder/isa/fastdctlee.py b/src/openpower/decoder/isa/fastdctlee.py index 15e3923c..d6647ac1 100644 --- a/src/openpower/decoder/isa/fastdctlee.py +++ b/src/openpower/decoder/isa/fastdctlee.py @@ -173,13 +173,12 @@ def transform2(vec): # actually: swap the *indices*... not the actual data. # incredibly... bizarrely... this works *without* having # to do anything else. - if len(j) > 1: - hz2 = halfsize // 2 - for ci, (jl, jh) in enumerate(zip(j[:hz2], jr[:hz2])): - tmp = ri[jl+halfsize] - ri[jl+halfsize] = ri[jh] - ri[jh] = tmp - print (" swap", size, i, ri[jl+halfsize], ri[jh]) + hz2 = halfsize // 2 # can be zero which stops reversing 1-item lists + for ci, (jl, jh) in enumerate(zip(j[:hz2], jr[:hz2])): + tmp = ri[jl+halfsize] + ri[jl+halfsize] = ri[jh] + ri[jh] = tmp + print (" swap", size, i, ri[jl+halfsize], ri[jh]) size //= 2 print("post-swapped", ri) -- 2.30.2