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

Fix fsdb dumping for vcs with all #186

Merged
merged 2 commits into from
Jun 9, 2022

Conversation

bensampson5
Copy link
Contributor

@bensampson5 bensampson5 commented Jun 9, 2022

This extends the work on #183. I realized after testing a little further that it wasn't dumping some variables that I wanted. It would cover basic things but, for example, wouldn't dump the values for any unpacked real arrays. I didn't know this at first but apparently you have to add fsdb all=on when you run the sim executable (typically called simv for vcs) to have it dump all types of variables and other more complicated system verilog constructs. Anyway, these changes will fix that. Below is a screenshot of it working.
image

Some other notes/changes as I was working through this:

  • -debug compile arg is soon to be deprecated by Synopsys so I updated that to -debug_access (at least this is what my vcs 2020 tells me, it may in fact already be deprecated today)
  • Using -debug_access no longer requires a separate vpi flag as the vcs documentation says it's already automatically turned on when you specify -debug_access
  • I leaned on the side of turning on as many debug features as possible when waveforms are enabled because typically when you want to look at waveforms you want to see everything that is going on. The downside is simulation runtime when waveforms are enabled but it will still run fast when waveforms are disabled because all those extra debug features will be turned off.

@themperek themperek merged commit ec3a324 into themperek:master Jun 9, 2022
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.

2 participants