Pranith
Pranith
@odanrc Thaks for the review. I squashed all the commits into one. Not sure if this is can be done during merge?
> > I just want to double check that even though you saw there were no stats differences, did you make sure the BTB is being used in the tests?...
@powerjg Can you please point me to docs on how to run the CI tests locally? It seems some quick boot tests are failing.
@giactra Can you please review? This now passes CI tests and is ready for merge. Please let me know if you want me to squash the two commits together or...
@andysan Now that we have the tag based addressing in the cache, can you please review the 'tagMask' commit of this PR? We need tagMask for the BTB (which I...
> A better solution would be to deal with truncation in the btb indexing policy instead where the semantic information is available. That'd let users decide how many bits of...
@giactra @odanrc Can you please review this updated patch? It would be good to have it merged if there are no new comments.
> @pranith I cannot spot the differences between the previous version and the new version. This is probably because the commits were squashed together. Could you point to where we...
I fixed the issue @erin-le , please review.
@BobbyRBruce please find an example use in multisim script here: https://github.com/pranith/gem5/blob/e58b3594a91ea570622f8ab02fed0c4f209d7e6b/configs/example/gem5_library/checkpoints/simpoints-se-restore_arm_specrate2017.py#L331