summaryrefslogtreecommitdiff
path: root/src/arch/arm/isa/insts
diff options
context:
space:
mode:
authorGene Wu <Gene.Wu@arm.com>2010-08-23 11:18:40 -0500
committerGene Wu <Gene.Wu@arm.com>2010-08-23 11:18:40 -0500
commita9931880348d3194be9cbb201a19b52d53d7ee83 (patch)
treedb08d170bfdea1add0c6c459e7d02ddcee8b0704 /src/arch/arm/isa/insts
parent0c434b7f5650be8e742199e2d1efb5d642e210c5 (diff)
downloadgem5-a9931880348d3194be9cbb201a19b52d53d7ee83.tar.xz
ARM: Temporary local variables can't conflict with isa parser operands.
PC is an operand, so we can't have a temp called PC
Diffstat (limited to 'src/arch/arm/isa/insts')
-rw-r--r--src/arch/arm/isa/insts/branch.isa30
1 files changed, 15 insertions, 15 deletions
diff --git a/src/arch/arm/isa/insts/branch.isa b/src/arch/arm/isa/insts/branch.isa
index 1aa37f483..089a2e7d9 100644
--- a/src/arch/arm/isa/insts/branch.isa
+++ b/src/arch/arm/isa/insts/branch.isa
@@ -46,16 +46,16 @@ let {{
# B, BL
for (mnem, link) in (("b", False), ("bl", True)):
bCode = '''
- Addr PC = readPC(xc);
- NPC = ((PC + imm) & mask(32)) | (PC & ~mask(32));
+ Addr curPc = readPC(xc);
+ NPC = ((curPc + imm) & mask(32)) | (curPc & ~mask(32));
'''
if (link):
bCode += '''
- Addr tBit = PC & (ULL(1) << PcTBitShift);
+ Addr tBit = curPc & (ULL(1) << PcTBitShift);
if (!tBit)
- LR = PC - 4;
+ LR = curPc - 4;
else
- LR = PC | 1;
+ LR = curPc | 1;
'''
bIop = InstObjParams(mnem, mnem.capitalize(), "BranchImmCond",
@@ -67,8 +67,8 @@ let {{
# BX, BLX
blxCode = '''
- Addr PC = readPC(xc);
- Addr tBit = PC & (ULL(1) << PcTBitShift);
+ Addr curPc = readPC(xc);
+ Addr tBit = curPc & (ULL(1) << PcTBitShift);
bool arm = !tBit;
arm = arm; // In case it's not used otherwise.
%(link)s
@@ -86,7 +86,7 @@ let {{
Name += "Imm"
# Since we're switching ISAs, the target ISA will be the opposite
# of the current ISA. !arm is whether the target is ARM.
- newPC = '(!arm ? (roundDown(PC, 4) + imm) : (PC + imm))'
+ newPC = '(!arm ? (roundDown(curPc, 4) + imm) : (curPc + imm))'
base = "BranchImm"
declare = BranchImmDeclare
constructor = BranchImmConstructor
@@ -102,23 +102,23 @@ let {{
// is 32 bits wide, but "next pc" doesn't reflect that
// so we don't want to substract 2 from it at this point
if (arm)
- LR = PC - 4;
+ LR = curPc - 4;
else
- LR = PC | 1;
+ LR = curPc | 1;
'''
elif link:
linkStr = '''
if (arm)
- LR = PC - 4;
+ LR = curPc - 4;
else
- LR = (PC - 2) | 1;
+ LR = (curPc - 2) | 1;
'''
else:
linkStr = ""
if imm and link: #blx with imm
branchStr = '''
- Addr tempPc = ((%(newPC)s) & mask(32)) | (PC & ~mask(32));
+ Addr tempPc = ((%(newPC)s) & mask(32)) | (curPc & ~mask(32));
FNPC = tempPc ^ (ULL(1) << PcTBitShift);
'''
else:
@@ -140,8 +140,8 @@ let {{
#CBNZ, CBZ. These are always unconditional as far as predicates
for (mnem, test) in (("cbz", "=="), ("cbnz", "!=")):
code = '''
- Addr PC = readPC(xc);
- NPC = ((PC + imm) & mask(32)) | (PC & ~mask(32));
+ Addr curPc = readPC(xc);
+ NPC = ((curPc + imm) & mask(32)) | (curPc & ~mask(32));
'''
predTest = "Op1 %(test)s 0" % {"test": test}
iop = InstObjParams(mnem, mnem.capitalize(), "BranchImmReg",