Review Board 2.0.15


cpu: Remove incorrect branch predictor drain checks

Review Request #1692 - Created Feb. 6, 2013 and updated

Information
Andreas Sandberg
gem5
default
Reviewers
Default 
Changeset 9494:f490aefa53d3
---------------------------
cpu: Remove incorrect branch predictor drain checks

The branch predictor unit incorrectly checked that the prediction
history was empty when the system is drained. This is an incorrect
assumption since the predictor isn't affected by a drain.

Compiles. The ARM realview-switcheroo-full test case works.
Ship it!
Ali Saidi
Posted (Feb. 7, 2013, 7:36 a.m.)
Ship It!
Nilay Vaish
Posted (Feb. 8, 2013, 7:42 a.m.)
Wasn't the switchroo test working before as well? IIRC it is part of the regression tests.
  1. It was. I have no idea why the tests didn't break, but I intend to look into it as soon as I get some slack. Unfortunately, I probably won't have time to look into it until after HPCA.