parsa-epfl / flexus Goto Github PK

In Trace mode, we have two stats for counting the number of executed instructions, but they do not match.
Fetches is updated in insn_fetch[here] and seems to be right, but ICount is based on the value that is returned through the API [here] and seems to be wrong.

Code line(s) affected:

Behaviour:

1. A conditional branch's retirement effects execute.
2. The feedback events are sent to the branch predictor in cycle.cpp:302 as above.
3. The branch then reaches the head of the SRB to commit, and resynchronizes with QEMU due to a mispredicted direction.
4. The branch predictor has already trained itself on the incorrect direction or target.

Upon calling getMemoryMap(), the control flow goes through the (pre-initialized) factory, which resolves to the function virtual boost::intrusive_ptr<MemoryMap> createMemoryMap() implemented by the MemoryMap component.
Creating a dummy MemoryMap instance initializes the factory. Although this works, it is not elegant. There no need for a MemoryMap instance. All we want is an initialized factory.
The right way of doing this for unit testing would be to create a mock component called MockMemoryMap which just implements wrappers for two functions that are actually used by MemoryLoopbackImpl:

• createMemoryMap
• recordAccess

The dummy MemoryMap instance needs to be removed and replaced by MockMemoryMap.