Andreas Sandberg has uploaded a new patch set (#3) to the change originally
created by Curtis Dunham. ( https://gem5-review.googlesource.com/2702 )
Change subject: cpu: Simplify the rename interface and use RegId
......................................................................
cpu: Simplify the rename interface and use RegId
With the hierarchical RegId there are a lot of functions that are
redundant now.
The idea behind the simplification is that instead of having the regId,
telling which kind of register read/write/rename/lookup/etc. and then
the function panic_if'ing if the regId is not of the appropriate type,
we provide an interface that decides what kind of register to read
depending on the register type of the given regId.
Change-Id: I7d52e9e21fc01205ae365d86921a4ceb67a57178
Reviewed-by: Andreas Sandberg <[email protected]>
[ Fix RISCV build issues ]
Signed-off-by: Andreas Sandberg <[email protected]>
---
M src/arch/alpha/isa.hh
M src/arch/alpha/isa/branch.isa
M src/arch/alpha/isa/fp.isa
M src/arch/alpha/isa/main.isa
M src/arch/arm/insts/misc.cc
M src/arch/arm/isa.hh
M src/arch/arm/isa/insts/data64.isa
M src/arch/arm/isa/insts/fp.isa
M src/arch/arm/isa/insts/misc.isa
M src/arch/mips/isa.hh
M src/arch/mips/isa/base.isa
M src/arch/mips/isa/formats/int.isa
M src/arch/power/insts/branch.cc
M src/arch/power/insts/static_inst.cc
M src/arch/power/isa.hh
M src/arch/riscv/isa.hh
M src/arch/riscv/isa/base.isa
M src/arch/riscv/isa/formats/type.isa
M src/arch/sparc/isa.hh
M src/arch/sparc/isa/base.isa
M src/arch/sparc/isa/formats/integerop.isa
M src/arch/sparc/isa/formats/mem/util.isa
M src/arch/sparc/isa/formats/priv.isa
M src/arch/x86/insts/microfpop.hh
M src/arch/x86/insts/microldstop.hh
M src/arch/x86/insts/micromediaop.hh
M src/arch/x86/insts/microregop.hh
M src/arch/x86/insts/static_inst.cc
M src/arch/x86/insts/static_inst.hh
M src/arch/x86/isa.hh
M src/arch/x86/isa/microops/limmop.isa
M src/cpu/base_dyn_inst.hh
M src/cpu/checker/cpu.hh
M src/cpu/checker/cpu_impl.hh
M src/cpu/checker/thread_context.hh
M src/cpu/exec_context.hh
M src/cpu/minor/dyn_inst.cc
M src/cpu/minor/exec_context.hh
M src/cpu/minor/scoreboard.cc
M src/cpu/minor/scoreboard.hh
M src/cpu/o3/comm.hh
M src/cpu/o3/cpu.cc
M src/cpu/o3/dyn_inst.hh
M src/cpu/o3/free_list.hh
M src/cpu/o3/iew_impl.hh
M src/cpu/o3/inst_queue_impl.hh
M src/cpu/o3/probe/elastic_trace.cc
M src/cpu/o3/regfile.cc
M src/cpu/o3/regfile.hh
M src/cpu/o3/rename.hh
M src/cpu/o3/rename_impl.hh
M src/cpu/o3/rename_map.cc
M src/cpu/o3/rename_map.hh
M src/cpu/o3/scoreboard.hh
M src/cpu/o3/thread_context.hh
M src/cpu/o3/thread_context_impl.hh
M src/cpu/reg_class.cc
M src/cpu/reg_class.hh
A src/cpu/reg_class_impl.hh
M src/cpu/simple/exec_context.hh
M src/cpu/simple_thread.hh
M src/cpu/static_inst.hh
M src/cpu/thread_context.hh
M src/cpu/timing_expr.cc
64 files changed, 741 insertions(+), 749 deletions(-)
--
To view, visit https://gem5-review.googlesource.com/2702
To unsubscribe, visit https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7d52e9e21fc01205ae365d86921a4ceb67a57178
Gerrit-Change-Number: 2702
Gerrit-PatchSet: 3
Gerrit-Owner: Curtis Dunham <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Anthony Gutierrez <[email protected]>
Gerrit-Reviewer: Curtis Dunham <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
