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

Parameters outside modules not recognized #471

Closed
veripoolbot opened this issue Mar 27, 2012 · 4 comments
Closed

Parameters outside modules not recognized #471

veripoolbot opened this issue Mar 27, 2012 · 4 comments
Assignees

Comments

@veripoolbot
Copy link
Collaborator

@veripoolbot veripoolbot commented Mar 27, 2012


Author Name: Corey Teffetalor
Original Redmine Issue: 471 from https://www.veripool.org
Original Date: 2012-03-27
Original Assignee: Wilson Snyder (@wsnyder)


It seems that "parameter" statements outside of module scope are not recognized by the Verilog-Perl Verilog::Netlist class. This seems to be is rare-but-valid syntax.

Would it be possible to add an interface to Verilog::Netlist to retrieve these global scoped parameters, or assign the "global" parameter net to all Verilog::Module in the Verilog::Netlist structure they apply to. I do not see them stored anywhere in the Verilog::Netlist structures using Data::Dumper.

This seems to have previously been noted at (2nd comment down):
http://www.veripool.org/boards/18/topics/show/410?r=575#message-575

module A( B, C );
input A;
output B;
parameter D = 1; // Can see this.
assign C = A;
endmodule

parameter D = 1; // Can't see this.
module A( B, C );
input A;
output B;
assign C = A;
endmodule

@veripoolbot

This comment has been minimized.

Copy link
Collaborator Author

@veripoolbot veripoolbot commented Mar 27, 2012


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2012-03-27T23:40:55Z


It parses them, just doesn't have a place to put them, as you indicated. I'll make a $root, which is where SystemVerilog says they go.

@veripoolbot

This comment has been minimized.

Copy link
Collaborator Author

@veripoolbot veripoolbot commented Mar 28, 2012


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2012-03-28T23:52:06Z


This is fixed in git towards 3.315.

@veripoolbot

This comment has been minimized.

Copy link
Collaborator Author

@veripoolbot veripoolbot commented Apr 9, 2012


Original Redmine Comment
Author Name: Corey Teffetalor
Original Date: 2012-04-09T19:05:35Z


Glanced at the git code. Thanks for the quick fix! Should be a good solution to retrieving these parameters.

@veripoolbot

This comment has been minimized.

Copy link
Collaborator Author

@veripoolbot veripoolbot commented May 4, 2012


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2012-05-04T21:50:57Z


In 3.315.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.