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

Jump to bottom

Can't trace signals with leading underscores. #280

Closed
veripoolbot opened this issue Aug 28, 2010 · 4 comments
Closed

Can't trace signals with leading underscores. #280

veripoolbot opened this issue Aug 28, 2010 · 4 comments
Assignees
@wsnyder
Labels
area: invoking/options Issue involves options passed to Verilator resolution: fixed Closed; fixed

Comments

@veripoolbot
Copy link
Contributor

Author Name: Jason McMullan (@ezrec)
Original Redmine Issue: 280 from https://www.veripool.org

Original Assignee: Wilson Snyder (@wsnyder)

I'm working with an existing (large) design that has a common idiom
of using an underscore ('_') prefix for wires/regs that are
active low.

Current head-of-line Verilator does not emit these signals in the
VCD trace, but does emit all other signals.

Is there an option to set (or a patch I can apply) to permit
Verilator to trace and dump signals that start with an underscore?
@veripoolbot
Copy link
Contributor Author

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2010-08-29T00:12:28Z

Would you like to attempt adding an option, say -trace-underscore, yourself?

Search for "Leading Underscore" and the next 3 lines in V3Trace.cpp. Then add a accessor to V3Option.h and parse it in V3Option.cpp, and document in bin/verilator. The options are in alphabetical order.

@veripoolbot
Copy link
Contributor Author

Original Redmine Comment
Author Name: Jason McMullan (@ezrec)
Original Date: 2010-08-29T15:54:36Z

Patch attached, versus verilator_3_803

@veripoolbot
Copy link
Contributor Author

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2010-08-29T23:28:54Z

Perfect! I forgot about coverage, glad you added that too.

Pushed to git for next release.

I may make tracing _'s the default, but want to wait a few releases for this switch to exist so people can migrate.

@veripoolbot
Copy link
Contributor Author

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2010-09-20T19:54:30Z

In 3.804.

@veripoolbot veripoolbot added area: invoking/options Issue involves options passed to Verilator resolution: fixed Closed; fixed labels Dec 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wsnyder wsnyder

Labels
area: invoking/options Issue involves options passed to Verilator resolution: fixed Closed; fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wsnyder @veripoolbot