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

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

Comments

@veripoolbot
Copy link

@veripoolbot veripoolbot commented Nov 26, 2014


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

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Nov 26, 2014


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

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Dec 20, 2014


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

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Dec 20, 2014


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


In 3.868.

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