cpu-o3:update MBTB entry resolution logic in test

Cao Jiaming · Cao Jiaming · commit 043f6d7ecedd · 2025-12-24T11:56:58.000+08:00
Change-Id: I2bc707fefe6648874a3c12190e669bd0933c9e7f
diff --git a/src/cpu/pred/btb/mbtb.cc b/src/cpu/pred/btb/mbtb.cc
@@ -726,7 +726,7 @@ MBTB::prepareUpdateEntries(const FetchStream &stream) {
     // Filter: only keep conditional branches that are not always taken
     if (getResolvedUpdate()) {
         auto remove_it = std::remove_if(all_entries.begin(), all_entries.end(),
-            [](const BTBEntry &e) { return !( e.resolved); });
+            [](const BTBEntry &e) { return !e.resolved; });
         all_entries.erase(remove_it, all_entries.end());
     }
 
diff --git a/src/cpu/pred/btb/test/btb.test.cc b/src/cpu/pred/btb/test/btb.test.cc
@@ -132,6 +132,10 @@ predictUpdateCycle(MBTB* btb,
         btb->getAndSetNewBTBEntry(stream);
     }
 
+    for (auto &entry : stream.updateBTBEntries) {
+        entry.resolved = true;
+    }
+
     btb->update(stream);
 
     // Return final predictions after update

Original file line number	Diff line number	Diff line change
`@@ -726,7 +726,7 @@ MBTB::prepareUpdateEntries(const FetchStream &stream) {`
`726`	`726`	`// Filter: only keep conditional branches that are not always taken`
`727`	`727`	`if (getResolvedUpdate()) {`
`728`	`728`	`auto remove_it = std::remove_if(all_entries.begin(), all_entries.end(),`
`729`		`- [](const BTBEntry &e) { return !( e.resolved); });`
	`729`	`+ [](const BTBEntry &e) { return !e.resolved; });`
`730`	`730`	`all_entries.erase(remove_it, all_entries.end());`
`731`	`731`	`}`
`732`	`732`
Original file line number	Diff line number	Diff line change
`@@ -132,6 +132,10 @@ predictUpdateCycle(MBTB* btb,`
`132`	`132`	`btb->getAndSetNewBTBEntry(stream);`
`133`	`133`	`}`
`134`	`134`
	`135`	`+ for (auto &entry : stream.updateBTBEntries) {`
	`136`	`+ entry.resolved = true;`
	`137`	`+ }`
	`138`	`+`
`135`	`139`	`btb->update(stream);`
`136`	`140`
`137`	`141`	`// Return final predictions after update`