cpu: remove faulty simpoint basic block inst count assertion

This patch removes an assertion in the simpoint profiling code that
asserts that a previously-seen basic block has the exact same number
of instructions executed as before. This can be false if the basic
block generates aborts or takes interrupts at different locations
within the basic block. The basic block profiling are not affected
significantly as these events are rare in general.

diff --git a/src/cpu/simple/atomic.cc b/src/cpu/simple/atomic.cc
index d29903c2f5ae6121b01dd36350d58fadbdafc045..13c4b9bd3c77c78e905f2c68ffa3e3e0cfb69fcf 100644 (file)
--- a/src/cpu/simple/atomic.cc
+++ b/src/cpu/simple/atomic.cc
@@ -618,7 +618,6 @@ AtomicSimpleCPU::profileSimPoint()
             // If basic block is seen before, just increment the count by the
             // number of insts in basic block.
             BBInfo& info = map_itr->second;
-            assert(info.insts == currentBBVInstCount);
             info.count += currentBBVInstCount;
         }
         currentBBVInstCount = 0;