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

Add more detailed statistics #851

Closed
veripoolbot opened this issue Nov 26, 2014 · 3 comments
Closed

Add more detailed statistics #851

veripoolbot opened this issue Nov 26, 2014 · 3 comments
Labels
resolution: fixed Closed; fixed

Comments

@veripoolbot
Copy link
Contributor

Author Name: Jeremy Bennett (@jeremybennett)
Original Redmine Issue: 851 from https://www.veripool.org
Original Date: 2014-11-26
Original Assignee: Jeremy Bennett (@jeremybennett)

When optimizing a model, it is useful not just to know how many vars of different sizes there are, but what those vars are. Large ones are candidates for splitting up.

The following patch adds a new flag @--extra-stats@ (which implies @--stats@), which will extend the vars section of the stats output to print the name of each variable of each size.

Please review and pull the patch from branch extra-stats at https://github.com/jeremybennett/verilator. This includes the patch of issue 849, since I needed this to get the system to compile.
@veripoolbot
Copy link
Contributor Author

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2014-11-26T21:56:16Z

I'd prefer to either call this -stats-vars, or --statsi 2 which is like the debugi and dump-treei options. It sort of depends how many similar features may get added here. Which do you prefer?

@veripoolbot
Copy link
Contributor Author

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2014-12-20T13:31:02Z

Went with --stats-vars. Pushed to git towards 3.857.

@veripoolbot
Copy link
Contributor Author

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2014-12-20T21:52:57Z

In 3.868.

@veripoolbot veripoolbot added the resolution: fixed Closed; fixed label Dec 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
resolution: fixed Closed; fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@veripoolbot