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

Bus indices should (arguably) be signed not unsigned #11

Open
gatecat opened this issue Feb 24, 2021 · 1 comment
Open

Bus indices should (arguably) be signed not unsigned #11

gatecat opened this issue Feb 24, 2021 · 1 comment

Comments

@gatecat
Copy link
Collaborator

gatecat commented Feb 24, 2021

Based on some recent comments in YosysHQ/yosys#2567 it is apparent that negative bus indices are legal Verilog (and occasionally used for the fractional part of fixed point values) and therefore should be representable in the interchange format too. This would mean using a signed integer type for these.

@litghost
Copy link
Contributor

Sure, this make sense.

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

No branches or pull requests

2 participants