Trivikram Reddy has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/25625 )
Change subject: cpu: change the location of BTBlookup
......................................................................
cpu: change the location of BTBlookup
BTBlookup should be done only if BTB is used, previously
this stat was updated for indirector predictor as well.
https: //gem5.atlassian.net/browse/GEM5-338
Change-Id: I20695dc7a8677d4fd0c4ae9f4f7d279387d5ad62
---
M src/cpu/pred/bpred_unit.cc
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/cpu/pred/bpred_unit.cc b/src/cpu/pred/bpred_unit.cc
index a2faad1..3d9e3ea 100644
--- a/src/cpu/pred/bpred_unit.cc
+++ b/src/cpu/pred/bpred_unit.cc
@@ -233,7 +233,6 @@
"RAS predicted target: %s, RAS index: %i\n",
tid, seqNum, pc, target, predict_record.RASIndex);
} else {
- ++BTBLookups;
if (inst->isCall()) {
RAS[tid].push(pc);
@@ -250,6 +249,7 @@
}
if (inst->isDirectCtrl() || !iPred) {
+ ++BTBLookups;
// Check BTB on direct branches
if (BTB.valid(pc.instAddr(), tid)) {
++BTBHits;
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/25625
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I20695dc7a8677d4fd0c4ae9f4f7d279387d5ad62
Gerrit-Change-Number: 25625
Gerrit-PatchSet: 1
Gerrit-Owner: Trivikram Reddy <tvre...@ucdavis.edu>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev
