Hello Ilias Vougioukas, Sudhanshu Jha, Andreas Sandberg,
I'd like you to reexamine a change. Please visit
https://gem5-review.googlesource.com/c/public/gem5/+/14216
to look at the new patch set (#3).
Change subject: cpu: Fixes on the loop predictor part of LTAGE
......................................................................
cpu: Fixes on the loop predictor part of LTAGE
Fixed the following fields of the loop predictor entries as described on
the LTAGE paper:
- Age counter (it was 3 bits and it should be 8 bits)
- Tag (it was 16 bits and it should be 14 bits). Also some times it used
int variables and some times uint16_t, leading to wrong behaviour
- Confidence counter (it was 2 bits ins some parts of the code and 3 bits
in some other parts. It should be 2 bits)
- Iteration counters (they were 16 bits and they should be 14 bits)
All the new sizes are now configurable
Change-Id: I8884c7454c1e510b65160eb4d5749d3259d34096
Signed-off-by: Pau Cabre <[email protected]>
---
M src/cpu/pred/BranchPredictor.py
M src/cpu/pred/ltage.cc
M src/cpu/pred/ltage.hh
3 files changed, 61 insertions(+), 11 deletions(-)
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/14216
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I8884c7454c1e510b65160eb4d5749d3259d34096
Gerrit-Change-Number: 14216
Gerrit-PatchSet: 3
Gerrit-Owner: Pau Cabre <[email protected]>
Gerrit-Assignee: Ilias Vougioukas <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Ilias Vougioukas <[email protected]>
Gerrit-Reviewer: Pau Cabre <[email protected]>
Gerrit-Reviewer: Sudhanshu Jha <[email protected]>
Gerrit-CC: Jason Lowe-Power <[email protected]>
Gerrit-MessageType: newpatchset
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
