diff options
author | Gabe Black <gblack@eecs.umich.edu> | 2010-06-02 12:58:01 -0500 |
---|---|---|
committer | Gabe Black <gblack@eecs.umich.edu> | 2010-06-02 12:58:01 -0500 |
commit | deb6e8f805e162dd6a33b154e6a635acc0d7c132 (patch) | |
tree | 3dd73c54340ddea754e978de596a9f19dee77e2a /src/arch/arm | |
parent | 190502476681b972f65697a7d2b3e3da35d40cd1 (diff) | |
download | gem5-deb6e8f805e162dd6a33b154e6a635acc0d7c132.tar.xz |
ARM: Eliminate the old memory formats which are no longer used.
Diffstat (limited to 'src/arch/arm')
-rw-r--r-- | src/arch/arm/insts/mem.cc | 19 | ||||
-rw-r--r-- | src/arch/arm/insts/mem.hh | 103 | ||||
-rw-r--r-- | src/arch/arm/isa/formats/mem.isa | 63 | ||||
-rw-r--r-- | src/arch/arm/isa/formats/util.isa | 36 |
4 files changed, 0 insertions, 221 deletions
diff --git a/src/arch/arm/insts/mem.cc b/src/arch/arm/insts/mem.cc index f62786979..4d56b80f8 100644 --- a/src/arch/arm/insts/mem.cc +++ b/src/arch/arm/insts/mem.cc @@ -67,23 +67,4 @@ MemoryNew::printInst(std::ostream &os, AddrMode addrMode) const } } -std::string -Memory::generateDisassembly(Addr pc, const SymbolTable *symtab) const -{ - std::stringstream ss; - printMnemonic(ss); - printReg(ss, machInst.rd); - ss << ", ["; - printReg(ss, machInst.rn); - ss << ", "; - if (machInst.puswl.prepost == 1) - printOffset(ss); - ss << "]"; - if (machInst.puswl.prepost == 0) - printOffset(ss); - else if (machInst.puswl.writeback) - ss << "!"; - return ss.str(); -} - } diff --git a/src/arch/arm/insts/mem.hh b/src/arch/arm/insts/mem.hh index 3c7a50505..dd6526fe1 100644 --- a/src/arch/arm/insts/mem.hh +++ b/src/arch/arm/insts/mem.hh @@ -229,109 +229,6 @@ class MemoryNewPostIndex : public Base return ss.str(); } }; - -/** - * Base class for general Arm memory-format instructions. - */ -class Memory : public PredOp -{ - protected: - - /// Memory request flags. See mem_req_base.hh. - unsigned memAccessFlags; - - /// Displacement for EA calculation (signed). - int32_t disp; - int32_t disp8; - int32_t up; - int32_t hilo, - shift_size, - shift; - - /// Constructor - Memory(const char *mnem, ExtMachInst _machInst, OpClass __opClass) - : PredOp(mnem, _machInst, __opClass), - memAccessFlags(0), - disp(machInst.immed11_0), - disp8(machInst.immed7_0 << 2), - up(machInst.puswl.up), - hilo((machInst.immedHi11_8 << 4) | machInst.immedLo3_0), - shift_size(machInst.shiftSize), shift(machInst.shift) - { - } - - std::string - generateDisassembly(Addr pc, const SymbolTable *symtab) const; - - virtual void - printOffset(std::ostream &os) const - {} -}; - -class MemoryDisp : public Memory -{ - protected: - /// Constructor - MemoryDisp(const char *mnem, ExtMachInst _machInst, OpClass __opClass) - : Memory(mnem, _machInst, __opClass) - { - } - - void - printOffset(std::ostream &os) const - { - ccprintf(os, "#%#x", (machInst.puswl.up ? disp : -disp)); - } -}; - -class MemoryHilo : public Memory -{ - protected: - /// Constructor - MemoryHilo(const char *mnem, ExtMachInst _machInst, OpClass __opClass) - : Memory(mnem, _machInst, __opClass) - { - } - - void - printOffset(std::ostream &os) const - { - ccprintf(os, "#%#x", (machInst.puswl.up ? hilo : -hilo)); - } -}; - -class MemoryShift : public Memory -{ - protected: - /// Constructor - MemoryShift(const char *mnem, ExtMachInst _machInst, OpClass __opClass) - : Memory(mnem, _machInst, __opClass) - { - } - - void - printOffset(std::ostream &os) const - { - printShiftOperand(os); - } -}; - -class MemoryReg : public Memory -{ - protected: - /// Constructor - MemoryReg(const char *mnem, ExtMachInst _machInst, OpClass __opClass) - : Memory(mnem, _machInst, __opClass) - { - } - - void - printOffset(std::ostream &os) const - { - os << (machInst.puswl.up ? "+ " : "- "); - printReg(os, machInst.rm); - } -}; } #endif //__ARCH_ARM_INSTS_MEM_HH__ diff --git a/src/arch/arm/isa/formats/mem.isa b/src/arch/arm/isa/formats/mem.isa index 648220884..13309ec9b 100644 --- a/src/arch/arm/isa/formats/mem.isa +++ b/src/arch/arm/isa/formats/mem.isa @@ -39,50 +39,6 @@ // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. // // Authors: Gabe Black -// Stephen Hines - -//////////////////////////////////////////////////////////////////// -// -// Memory-format instructions -// - -def template LoadStoreDeclare {{ - /** - * Static instruction class for "%(mnemonic)s". - */ - class %(class_name)s : public %(base_class)s - { - public: - - /// Constructor. - %(class_name)s(ExtMachInst machInst); - - %(BasicExecDeclare)s - - %(InitiateAccDeclare)s - - %(CompleteAccDeclare)s - }; -}}; - - -def template InitiateAccDeclare {{ - Fault initiateAcc(%(CPU_exec_context)s *, Trace::InstRecord *) const; -}}; - - -def template CompleteAccDeclare {{ - Fault completeAcc(PacketPtr, %(CPU_exec_context)s *, Trace::InstRecord *) const; -}}; - - -def template LoadStoreConstructor {{ - inline %(class_name)s::%(class_name)s(ExtMachInst machInst) - : %(base_class)s("%(mnemonic)s", machInst, %(op_class)s) - { - %(constructor)s; - } -}}; def format AddrMode2(imm) {{ if eval(imm): @@ -514,22 +470,3 @@ def format Thumb16MemLit() {{ ''' % loadImmClassName(False, True, False) }}; -def format ArmLoadMemory(memacc_code, ea_code = {{ EA = Rn + disp; }}, - mem_flags = [], inst_flags = []) {{ - ea_code = ArmGenericCodeSubs(ea_code) - memacc_code = ArmGenericCodeSubs(memacc_code) - (header_output, decoder_output, decode_block, exec_output) = \ - LoadStoreBase(name, Name, ea_code, memacc_code, mem_flags, inst_flags, - decode_template = BasicDecode, - exec_template_base = 'Load') -}}; - -def format ArmStoreMemory(memacc_code, ea_code = {{ EA = Rn + disp; }}, - mem_flags = [], inst_flags = []) {{ - ea_code = ArmGenericCodeSubs(ea_code) - memacc_code = ArmGenericCodeSubs(memacc_code) - (header_output, decoder_output, decode_block, exec_output) = \ - LoadStoreBase(name, Name, ea_code, memacc_code, mem_flags, inst_flags, - exec_template_base = 'Store') -}}; - diff --git a/src/arch/arm/isa/formats/util.isa b/src/arch/arm/isa/formats/util.isa index d4dd41d2b..286ce5ce5 100644 --- a/src/arch/arm/isa/formats/util.isa +++ b/src/arch/arm/isa/formats/util.isa @@ -38,42 +38,6 @@ def ArmGenericCodeSubs(code): new_code = re.sub(r'Rm_Rs', 'shift_rm_rs(Rm, Rs, shift, CondCodes<29:>)', new_code) return new_code - -def LoadStoreBase(name, Name, ea_code, memacc_code, mem_flags, inst_flags, - base_class = 'Memory', - decode_template = BasicDecode, exec_template_base = ''): - # Make sure flags are in lists (convert to lists if not). - mem_flags = makeList(mem_flags) - inst_flags = makeList(inst_flags) - - iop = InstObjParams(name, Name, base_class, - {'ea_code': ea_code, - 'memacc_code': memacc_code, - 'predicate_test': predicateTest}, - inst_flags) - - if mem_flags: - s = '\n\tmemAccessFlags = ' + string.join(mem_flags, '|') + ';' - iop.constructor += s - memacc_iop.constructor += s - - # select templates - - # The InitiateAcc template is the same for StoreCond templates as the - # corresponding Store template.. - StoreCondInitiateAcc = StoreInitiateAcc - - fullExecTemplate = eval(exec_template_base + 'Execute') - initiateAccTemplate = eval(exec_template_base + 'InitiateAcc') - completeAccTemplate = eval(exec_template_base + 'CompleteAcc') - - # (header_output, decoder_output, decode_block, exec_output) - return (LoadStoreDeclare.subst(iop), - LoadStoreConstructor.subst(iop), - decode_template.subst(iop), - fullExecTemplate.subst(iop) - + initiateAccTemplate.subst(iop) - + completeAccTemplate.subst(iop)) }}; |