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

Add spike RISC-V simulator support #184

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

widlarizer
Copy link

As unhelpful it may be for performance testing, spike is a valid target for evaluating compilers on code size while ensuring the benchmarks are still functional. This directory has been glued together from various fragments of community code with original licenses ensured to be GPL compatible and copyright notices modified with SPDX tags added. Some open questions:

  • Is crt0 allowed to be present at all, with some added switch? The motivation is testing RISC-V LLVM
  • Should the sources be merged to be consistent, which would possibly harm manual and automated licensing compatibility reviews?

@widlarizer
Copy link
Author

Resulting time parsing isn't implemented yet due to riscv-software-src/riscv-isa-sim#1493

@widlarizer
Copy link
Author

That implemented cycle count parsing. The ISA is hard coded to RV32GC and letting that be configurable is another open question

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

Successfully merging this pull request may close these issues.

1 participant