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.

Sign up for GitHub

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

Jump to bottom

verilator-harness.cc broken due to (my) HTIF/FESVR API changes #12

Closed
seldridge opened this issue Jan 19, 2018 · 1 comment
Closed

verilator-harness.cc broken due to (my) HTIF/FESVR API changes #12

seldridge opened this issue Jan 19, 2018 · 1 comment

Comments

@seldridge
Copy link
Member

My changes to HTIF killed verilator-harness.cc (in addition to SimSerial.cc, but it looks like @zhemao already took care of that) and that's percolated into project-template: ucb-bar/chipyard#25.

Basically, verilator-harness.cc just needs to be modified to pass argc/argv to tsi_t. However, that could benefit from a refactor to be more like emulator.cc and get all the command line arguments out in the open.

If I have time, I'll try and take a look.
@colinschmidt
Copy link
Contributor

I think this was fixed with f8d7bed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@seldridge @colinschmidt