derek/gem5
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

arch-arm: python3 "/" will always produce a float

Browse Source 
"/" was ambiguous in python2 and was producing a floor (integer)
division if the operands were int or long.
In python3 "/" will always produce a float which makes it unsuitable
in cases where an integer is expected

PEP238: https://www.python.org/dev/peps/pep-0238/

Change-Id: I481cf1e9c0f95a6f47ecf6539eee0c9bcaf31e17
Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/26247
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>
Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
Giacomo Travaglini
2020-03-02 11:56:45 +00:00
parent 656b18a414
commit 41df2f8b26
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/arch/arm/isa/insts/macromem.isa
View File

@@ -580,7 +580,7 @@ let {{
exec_output = ''
for typeSize in (8, 16, 32):
for sRegs in 1, 2:
for dRegs in range(sRegs, min(sRegs * 64 / typeSize + 1, 5)):
for dRegs in range(sRegs, min(sRegs * 64 // typeSize + 1, 5)):
for format in ("MicroUnpackNeon%(sRegs)dto%(dRegs)dUop",
"MicroUnpackAllNeon%(sRegs)dto%(dRegs)dUop",
"MicroPackNeon%(dRegs)dto%(sRegs)dUop"):

2
src/arch/arm/isa/insts/neon64.isa
View File

@@ -552,7 +552,7 @@ let {{
%(op)s
destReg.elements[0] = htole(destElem);
''' % { "op" : op }
destCnt = rCount / 2
destCnt = rCount // 2
for reg in range(destCnt):
eWalkCode += '''
AA64FpDestP%(reg)d_uw = letoh(destReg.regs[%(reg)d]);