Skip to content
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

Address https://github.com/riscv/riscv-debug-spec/issues/985 #986

Closed
wants to merge 9 commits into from

Conversation

pdonahue-ventana
Copy link
Collaborator

I believe that this bug fix should be addressed before ratification.

/cc @en-sc

@rtwfroody
Copy link
Collaborator

Since the spec is frozen, I think we need to run this through architecture review. Can you rebase and submit this PR to be relative to rc2? Then we can include it in the batch of changes I was going to send over there today but haven't yet, so we can still squeeze this in.

@pdonahue-ventana
Copy link
Collaborator Author

I just rebased to rc2.

@pdonahue-ventana
Copy link
Collaborator Author

Well, that doesn't look right. Did you want me to rebase to rc2 and have the PR merge into rc2? I don't know how to change the target branch of the PR.

@pdonahue-ventana pdonahue-ventana changed the base branch from main to rc2 March 21, 2024 00:42
@pdonahue-ventana
Copy link
Collaborator Author

I think that I figured that out but now there are additional changes that we already dealt with in different PRs.

@rtwfroody
Copy link
Collaborator

I don't quite understand what's going on with this history. I made #987 which was easier than trying to figure this out.

@pdonahue-ventana
Copy link
Collaborator Author

OK. We can close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants