derek/gem5
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cpu: convert loop_predictor to new style stats

Browse Source 
Change-Id: Ib0383fc6d5f884fd6c020bcd938eee2f802ad412
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/33935
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
Emily Brickey
2020-09-01 14:02:58 -07:00
parent 343a62bab6
commit b4aefe4e86
2 changed files with 15 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
src/cpu/pred/loop_predictor.cc
View File

@@ -57,7 +57,8 @@ LoopPredictor::LoopPredictor(LoopPredictorParams *p)
restrictAllocation(p->restrictAllocation),
initialLoopIter(p->initialLoopIter),
initialLoopAge(p->initialLoopAge),
optionalAgeReset(p->optionalAgeReset)
optionalAgeReset(p->optionalAgeReset),
stats(this)
{
assert(initialLoopAge <= ((1 << loopTableAgeBits) - 1));
}
@@ -314,9 +315,9 @@ void
LoopPredictor::updateStats(bool taken, BranchInfo* bi)
{
if (taken == bi->loopPred) {
loopPredictorCorrect++;
stats.correct++;
} else {
loopPredictorWrong++;
stats.wrong++;
}
}
@@ -344,18 +345,13 @@ LoopPredictor::condBranchUpdate(ThreadID tid, Addr branch_pc, bool taken,
loopUpdate(branch_pc, taken, bi, tage_pred);
}
void
LoopPredictor::regStats()
LoopPredictor::LoopPredictorStats::LoopPredictorStats(Stats::Group *parent)
: Stats::Group(parent),
ADD_STAT(correct, "Number of times the loop predictor is"
" the provider and the prediction is correct"),
ADD_STAT(wrong, "Number of times the loop predictor is the"
" provider and the prediction is wrong")
{
loopPredictorCorrect
.name(name() + ".loopPredictorCorrect")
.desc("Number of times the loop predictor is the provider and "
"the prediction is correct");
loopPredictorWrong
.name(name() + ".loopPredictorWrong")
.desc("Number of times the loop predictor is the provider and "
"the prediction is wrong");
}
size_t

13
src/cpu/pred/loop_predictor.hh
View File

@@ -83,9 +83,11 @@ class LoopPredictor : public SimObject
const unsigned initialLoopAge;
const bool optionalAgeReset;
// stats
Stats::Scalar loopPredictorCorrect;
Stats::Scalar loopPredictorWrong;
struct LoopPredictorStats : public Stats::Group {
LoopPredictorStats(Stats::Group *parent);
Stats::Scalar correct;
Stats::Scalar wrong;
} stats;
/**
* Updates an unsigned counter based on up/down parameter
@@ -250,11 +252,6 @@ class LoopPredictor : public SimObject
*/
void init() override;
/**
* Register stats for this object
*/
void regStats() override;
LoopPredictor(LoopPredictorParams *p);
size_t getSizeInBits() const;