-
Notifications
You must be signed in to change notification settings - Fork 673
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
Bump Spike #1912
Bump Spike #1912
Conversation
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.
While we are at it, can you also update the SpikeTile ISA to have zvl128b_zve64d
and zicsr_zifencei_zihpm
(to match Rocket) (so something like this: rv64imafdvczicsr_zifencei_zihpm_zvl128b_zve64d
).
I also want these two PRs in for checkpointing: riscv-software-src/riscv-isa-sim#1715 riscv-software-src/riscv-isa-sim#1714 but I can re-bump later
Rocket does not currently have Zvl/Zve, are you sure you want them in SpikeTile? |
SpikeTiles current ISA is "rv64gcv_Zfh". I don't need V, but I figured you put V in there in the past to support it for some reason. |
Oh right. You are right then, we should put the full isa str |
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.
Thanks for the SpikeTile fix
Related PRs / Issues:
Type of change:
Impact:
Contributor Checklist:
main
as the base branch?changelog:<topic>
label?changelog:
label?.conda-lock.yml
file if you updated the conda requirements file?Please Backport
?