-
Notifications
You must be signed in to change notification settings - Fork 46
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
Reset ports for seq-mem
are ignored in reset-insertion
#1354
Comments
The warning is benign. Can you follow instruction for debugging compilation bugs to see if there is problem with any of the passes? If running with |
One interesting thing that does fix the problem: in
If we remove the So I think it's basically that the |
Oh weird, that just says that you shouldn't write anything to the memory while |
Sure; which wave viewer program should I use. Would just GTKWave (that's one of the ones listed in the documentation) be fine? |
If you use a Mac, you can use Scanscion too |
Seems like we're just never triggering the This is probably something for the TDST pass? Is there a specific place in the TDST code where I should look? |
Could be that I forgot to add a |
Huh, looks like I didn't. Can you look at the |
The pass skips cells with the |
@calebmkim I might've missed this but did you open a PR to fix this? |
seq-mem
are ignored in reset-insertion
I didn't because I read #1034 and I wasn't quite sure I fully understood the issue (maybe we can discuss more syncrhonously?). However, if all that's required is to make a PR to re-enable reset insertion for external cells, then I can definitely do that. |
If I run:
I get 220, in the "out" value, as expected.
If I run (same thing but thru icarus-verilog not verilog):
I get 0, in the "out" value, which is not expected.
Does this reproduce for other people?
The
iverilog
version is 11.0, which is fine according tofud check
. Another note is that I get a warning warning:when I run thru verilog.
The text was updated successfully, but these errors were encountered: