-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add hardware debug ROB #3560
Add hardware debug ROB #3560
Conversation
(Only for integer registers)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joey0320 we want to be able to remove the debugrob where necessary, please add that support to the WithRocketDebugROB.
Co-authored-by: Jerry Zhao <[email protected]>
Co-authored-by: Jerry Zhao <[email protected]>
Please update the title of this PR so that the merge commit will convey more useful information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, In the future, I'd like to migrate it to layer API for splitting RTL and verification codes
Related issue:
Type of change: bug report | feature request | other enhancement
Impact: no functional change | API addition (no impact on existing code) | API modification
Development Phase: proposal | implementation
Release Notes