Hello Jason Lowe-Power, Alec Roelke, Alec Roelke, Andreas Sandberg, Giacomo
Gabrielli,
I'd like you to reexamine a change. Please visit
https://gem5-review.googlesource.com/6821
to look at the new patch set (#4).
Change subject: alpha,arm,mips,power,riscv,sparc,x86,cpu: Get rid of
ISA_HAS_DELAY_SLOT.
......................................................................
alpha,arm,mips,power,riscv,sparc,x86,cpu: Get rid of ISA_HAS_DELAY_SLOT.
This constant is, first, a #define, and second only used in one place.
The O3 CPU is complex enough that a small hint that an if can be
skipped likely won't have any impact on performance. By getting rid of
this constant, we remove one small way non-ISA components are ISA
specific.
Change-Id: I820ebc9df10aeb4fcb69c12f6a784e9ec616743c
---
M src/arch/alpha/isa_traits.hh
M src/arch/arm/isa_traits.hh
M src/arch/mips/isa_traits.hh
M src/arch/power/isa_traits.hh
M src/arch/riscv/isa_traits.hh
M src/arch/sparc/isa_traits.hh
M src/arch/x86/isa_traits.hh
M src/cpu/o3/fetch_impl.hh
8 files changed, 1 insertion(+), 22 deletions(-)
--
To view, visit https://gem5-review.googlesource.com/6821
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I820ebc9df10aeb4fcb69c12f6a784e9ec616743c
Gerrit-Change-Number: 6821
Gerrit-PatchSet: 4
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-Reviewer: Alec Roelke <[email protected]>
Gerrit-Reviewer: Alec Roelke <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Giacomo Gabrielli <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-CC: Andreas Hansson <[email protected]>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
