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

Incorrect result for packed array with non-zero LSB typedef #1172

Closed
veripoolbot opened this issue Jun 5, 2017 · 3 comments
Closed

Incorrect result for packed array with non-zero LSB typedef #1172

veripoolbot opened this issue Jun 5, 2017 · 3 comments

Comments

@veripoolbot
Copy link

@veripoolbot veripoolbot commented Jun 5, 2017


Author Name: James Pallister
Original Redmine Issue: 1172 from https://www.veripool.org

Original Assignee: Wilson Snyder (@wsnyder)


Hi,

Defining an array with a type created by a typedef, then using that array with a subscript results in the wrong value.

I've added a testcase to show the problem at: https://github.com/jpallister/verilator on branch typedef_array

Cheers, James

@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Jun 7, 2017


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2017-06-07T00:07:40Z


Fixed in git towards 3.905.

The problem was more related to the :1 in the unpacked array. I fixed another similar problem with little endian arrays. There might be additional issues, so watch for that.

@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Jun 7, 2017


Original Redmine Comment
Author Name: James Pallister
Original Date: 2017-06-07T10:31:41Z


Thats great, thanks!

@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Jun 22, 2017


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2017-06-22T22:37:47Z


In 3.906.

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.