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

hdl.ast: Fix width for unary minus operator on signed argument. #279

Merged
merged 1 commit into from Dec 4, 2019
Merged

hdl.ast: Fix width for unary minus operator on signed argument. #279

merged 1 commit into from Dec 4, 2019

Conversation

mwkmwkmwk
Copy link
Contributor

To properly represent a negation of a signed X-bit quantity we may, in
general, need a signed (X+1)-bit signal — for example, negation of
3-bit -4 is 4, which is not representable in signed 3 bits.

To properly represent a negation of a signed X-bit quantity we may, in
general, need a signed (X+1)-bit signal — for example, negation of
3-bit -4 is 4, which is not representable in signed 3 bits.
@codecov
Copy link

codecov bot commented Dec 3, 2019

Codecov Report

Merging #279 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #279      +/-   ##
==========================================
- Coverage   82.19%   82.16%   -0.04%     
==========================================
  Files          34       34              
  Lines        5645     5645              
  Branches     1161     1160       -1     
==========================================
- Hits         4640     4638       -2     
- Misses        860      862       +2     
  Partials      145      145
Impacted Files Coverage Δ
nmigen/hdl/ast.py 87.19% <100%> (-0.23%) ⬇️
nmigen/build/plat.py 26.63% <0%> (ø) ⬆️
nmigen/back/rtlil.py 83.41% <0%> (ø) ⬆️
nmigen/lib/fifo.py 90.82% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7650431...2f6d3c6. Read the comment docs.

@whitequark whitequark merged commit 6765021 into m-labs:master Dec 4, 2019
@whitequark
Copy link
Contributor

Thank you!

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

Successfully merging this pull request may close these issues.

None yet

2 participants