-
Notifications
You must be signed in to change notification settings - Fork 415
Import PicoSoC as a VtR benchmark #582
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
Comments
FYI - @acomodi - Do you want to take this on, or is it better for @vaughnbetz to get one of his students to look at this? |
One potential issue is that most benchmarks in VtR are using ODIN-II, while SymbiFlow uses Yosys. |
If @acomodi can try pushing it through Odin-II that would be great. If there's no error please we can add it to the VTR benchmarks. If there is an error, please file an issue on it. If this doesn't go through Odin-II and there's no immediate fix, we could still add it as a benchmark, and use yosys for synthesis. Once we know what flows this benchmark goes through, and we can figure out how to add it. If it's a single file and it goes through Odin-II we can just add it to /vtr_flow/benchmarks/verilog and the appropriate regressions like vtr_flow/tasks/regression_tests/vtr_reg_nightly/vtr_reg_qor_chain/config/config.txt (add to circuit_list_add). If it only goes through Yosys and/or has multiple files we don't want to concantenate we can make a new regression to run it through its own flow. |
Adding @suyang5 as he is currently pushing new benchmarks through the flow, so he'll be interested in how this is done, and may also help out. |
On the Odin side of things, we might be able to get most of picoSoC through at first glance, I'll get someone on the ' inout '. And there's a couple of PR waiting to be merged. Hopefully that should get it in! |
@vaughnbetz @mithro Sure, I can take a look at this. |
@mithro Ok, I started to look into this today. When I have something reasonable to push I'll open the PR. |
I'd consider that a separate issue. We'd like to be able to use this benchmark with multiple architectures. #2: Add in the Artix7 and iCE40 architectures and make a regtest targeting this same benchmark at them. I am sure symbiflow already has such a regtest, so the question is whether we should also have one in the vtr source tree and continuous integration; seems like having one makes sense so we see right away if someone breaks these architectures. |
This issue has been inactive for a year and has been marked as stale. It will be closed in 15 days if it continues to be stale. If you believe this is still an issue, please add a comment. |
This issue has been marked stale for 15 days and has been automatically closed. |
The SymbiFlow project is using PicoSoC. This is a "Cortex M0" style 32bit RISC-V processor + supporting infrastructure (XIP SPI controller, UART, etc).
The SymbiFlow Architecture Definitions repository is already running PicoSoC through VtR with our own architecture files both for Xilinx Series 7 and Lattice iCE40. You can find a copy of the exact things we are testing here.
It would be good if this benchmark was included in the normal VtR benchmarks.
The text was updated successfully, but these errors were encountered: