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

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

Comments

@veripoolbot
Copy link

@veripoolbot veripoolbot commented Aug 28, 2010


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

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Aug 29, 2010


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

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Aug 29, 2010


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


Patch attached, versus verilator_3_803

@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Aug 29, 2010


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

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Sep 20, 2010


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


In 3.804.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.