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

Interpreter's reset behavior does not match verilator backend. #118

Open
chick opened this issue Feb 27, 2018 · 2 comments
Open

Interpreter's reset behavior does not match verilator backend. #118

chick opened this issue Feb 27, 2018 · 2 comments

Comments

@chick
Copy link
Contributor

chick commented Feb 27, 2018

The interpreter lower reset at a different point in the clock cycle than the verilator backend.
Although we have not strictly specified what our reset behavior is, it would be nicer if the two
agreed. Most test harnesses are not broken by this though it is not that hard to write a test that will be sensitive to the difference.

@chick
Copy link
Contributor Author

chick commented Feb 27, 2018

Note: The chisel ShiftResetTester test, when run with version of chisel that tests using the interpreter, seems to be exhibit this sensitivity

@seldridge
Copy link

I think there's a broader problem of the interpreter having x's in simulation. I got bit by this today due x-optimism. Both Treadle and the Verilator backends are doing the right thing and initializing everything.

(Note: I'm not really expecting a fix here, just that we should probably move everything off of using the FIRRTL interpreter by default as part of completely deprecating the interpreter.)

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

No branches or pull requests

2 participants