-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/1684/
-----------------------------------------------------------

Review request for Default.


Description
-------

The copyResult function is called when there is a result mismatch between the 
checker and the cpu for a load instruction (in most of the case, this mismatch 
is due to a store forwarding). This copyResult function calls the 
setFloatRegBits function that writes to the floating point registers of the 
checker. As the used index starts at theISA::FP_Base_DepTag, the assert 
assert(flatIndex < TheISA::NumFloatRegs) breaks. This patch substract the 
theISA::FP_Base_DepTag value to the index so that the assert does not break.


Diffs
-----

  src/cpu/checker/cpu_impl.hh 890fc69ba53cc95748f0ee51e4ccddb678019c5b 

Diff: http://reviews.gem5.org/r/1684/diff/


Testing
-------

It seems to correct the bug I had with the checker. However, I have not done 
extensive tests.


Thanks,

Nathanael Premillieu

_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to