-
Notifications
You must be signed in to change notification settings - Fork 217
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
RISC-V JALR: Handle cases where rd
==rs1
, and rd
!=zero
#6064
base: dev
Are you sure you want to change the base?
Conversation
So half of this PR was merged with #6213. Your other change, adding the unresolved branch to JALR, is being considered. I am not entirely sure if there was a reason why the branch was left out which is why I can't merge that part now. |
Talking to @rssor
If you look at #6064 (comment) you can see an example of the function calls. Looking at your comment #6003 (comment) it seems you are running into a different set of issues resulting from the omitted branch info, if you can share more information about what was fixed by this change maybe their is some alternative solution. |
Solves #6003