From c919ab5b4fa1d9f8721552dd2831c71c953c914f Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Wed, 2 Jun 2010 12:58:15 -0500 Subject: [PATCH] ARM: Fix double precision load/store multiple decrement. When decrementing, the higher addressed half of a double word is at a 4 byte smaller displacement. --- src/arch/arm/insts/macromem.cc | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/src/arch/arm/insts/macromem.cc b/src/arch/arm/insts/macromem.cc index 01ec3e1d8..e47f4c21c 100644 --- a/src/arch/arm/insts/macromem.cc +++ b/src/arch/arm/insts/macromem.cc @@ -161,7 +161,7 @@ MacroVFPMemOp::MacroVFPMemOp(const char *mnem, ExtMachInst machInst, numMicroops = count * (single ? 1 : 2) + (writeback ? 1 : 0); microOps = new StaticInstPtr[numMicroops]; - uint32_t addr = 0; + int64_t addr = 0; if (!up) addr = 4 * offset; @@ -172,21 +172,22 @@ MacroVFPMemOp::MacroVFPMemOp(const char *mnem, ExtMachInst machInst, microOps[i++] = new MicroLdrFpUop(machInst, vd++, rn, tempUp, addr); if (!single) - microOps[i++] = new MicroLdrFpUop(machInst, vd++, rn, - tempUp, addr + 4); + microOps[i++] = new MicroLdrFpUop(machInst, vd++, rn, tempUp, + addr + (up ? 4 : -4)); } else { microOps[i++] = new MicroStrFpUop(machInst, vd++, rn, tempUp, addr); if (!single) - microOps[i++] = new MicroStrFpUop(machInst, vd++, rn, - tempUp, addr + 4); + microOps[i++] = new MicroStrFpUop(machInst, vd++, rn, tempUp, + addr + (up ? 4 : -4)); } if (!tempUp) { addr -= (single ? 4 : 8); // The microops don't handle negative displacement, so turn if we // hit zero, flip polarity and start adding. - if (addr == 0) { + if (addr <= 0) { tempUp = true; + addr = -addr; } } else { addr += (single ? 4 : 8); -- 2.30.2