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

big bit-endian vs little bit-endian #36

Closed
veripoolbot opened this issue Oct 15, 2008 · 3 comments
Closed

big bit-endian vs little bit-endian #36

veripoolbot opened this issue Oct 15, 2008 · 3 comments
Labels
resolution: fixed Closed; fixed type: feature-IEEE Request to add new feature, described in IEEE 1800

Comments

@veripoolbot
Copy link
Contributor

Author Name: Rafael Shirakawa
Original Redmine Issue: 36 from https://www.veripool.org
Original Date: 2008-10-15

Hello,

is there any forecast to verilator accept little bit-endian?

regards,
Rafael
@veripoolbot
Copy link
Contributor Author

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2008-10-15T15:44:26Z

No as this is a very rare practice - do you work for IBM? ;) But if you want to fix it yourself, I'll point you to what to change and take the patches.

The big problem is C itself is big-bit-endian in that variable shift operations assume bit 0 is the LSB.
That's fixable by Verilator adding a subtraction at every shift, but the user would still need to know that the bits observed coming out of the model would be numbered in the C, not Verilog way.

@veripoolbot
Copy link
Contributor Author

Original Redmine Comment
Author Name: Rafael Shirakawa
Original Date: 2008-10-17T12:47:15Z

Ok, thanks for the indications... but I will take action in changing my code instead of Verilator's. :)

@veripoolbot
Copy link
Contributor Author

Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2012-03-13T01:03:54Z

Happened across this old bug, FYI little endian support was added in 3.720.

@veripoolbot veripoolbot added resolution: fixed Closed; fixed type: feature-IEEE Request to add new feature, described in IEEE 1800 labels Dec 22, 2019
@veripoolbot veripoolbot mentioned this issue Dec 22, 2019
Closed
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 type: feature-IEEE Request to add new feature, described in IEEE 1800
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@veripoolbot