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

Test for System Verilog enumeration methods #515

Closed
veripoolbot opened this issue May 16, 2012 · 3 comments
Closed

Test for System Verilog enumeration methods #515

veripoolbot opened this issue May 16, 2012 · 3 comments

Comments

@veripoolbot
Copy link

@veripoolbot veripoolbot commented May 16, 2012


Author Name: Jeremy Bennett (@jeremybennett)
Original Redmine Issue: 515 from https://www.veripool.org
Original Date: 2012-05-16
Original Assignee: Jeremy Bennett (@jeremybennett)


System Verilog provides a set of methods for operating on enumerations. This new test, @t_sv_enum_type_methods.pl@, exercises those methods.

Please pull the test from https://github.com/jeremybennett/verilator/tree/sv-enum-test.

@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented May 16, 2012


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2012-05-16T22:18:57Z


Thanks for the test.

This test needs to pass on VCS, NC or another simulator first. All complain "localparam t_pinid list = list.first;" is not considered a constant expression.

Also please remove trailing white space, and please add a $Self->unsupported call in the .pl file so the test passes until these features are supported.

@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented May 21, 2012


Original Redmine Comment
Author Name: Jeremy Bennett (@jeremybennett)
Original Date: 2012-05-21T12:23:22Z


Please pull an updated version from https://github.com/jeremybennett/verilator/tree/sv-enum-test.

The intention was to test the enumeration methods, so I have ditched the localparam. I will investigate whether any simulator supports that - I was led to believe that at least one treated these methods as constant.

The attached test passes with VCS. I have stripped all trailing space and added a $Self->unsupported call.

@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented May 21, 2012


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2012-05-21T22:42:54Z


Great, committed.

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.