On Sun, 12 Jul 2015, Joel Hestness wrote:
On July 12, 2015, 3:29 p.m., Joel Hestness wrote:
src/mem/ruby/system/Sequencer.cc, line 323
<http://reviews.gem5.org/r/2953/diff/1/?file=47948#file47948line323>
To keep things clean, could we also remove the clearLocked, isLocked, and
setLocked functions from the CacheMemory, since they are only used where you've
changed code?
Nilay Vaish wrote:
Somebody might be using them. It does not hurt us at all to leave them in
place.
Sure. In that case, could you add a comment near their declarations in
CacheMemory that the suggested functions for handling LL/SC are now in
CacheEntry?
That seems to be the right thing to do.
--
Nilay
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
