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.

Sign up for GitHub

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

SInt unary negation (expanding width) note in description is not consistent #2728

Closed
moriglia opened this issue Sep 14, 2022 · 0 comments · Fixed by #2729
Closed

SInt unary negation (expanding width) note in description is not consistent #2728

moriglia opened this issue Sep 14, 2022 · 0 comments · Fixed by #2729

Comments

@moriglia
Copy link
Contributor

In note for unary negation (with expanding width) documentation, the width is said to be unchanged.
"The width of the returned SInt is unchanged, i.e., width of this."

https://github.com/chipsalliance/chisel3/blob/255c56c3955a8c16191a6751e7d547cfcfd96705/core/src/main/scala/chisel3/Bits.scala#L902
@moriglia moriglia mentioned this issue Sep 14, 2022
Merged
16 tasks
jackkoenig pushed a commit that referenced this issue Sep 15, 2022
@moriglia
Change description for SInt unary negation (#2729)
a4dae9c 
Referenced to: #2728
mergify bot pushed a commit that referenced this issue Sep 15, 2022
@moriglia @mergify
Change description for SInt unary negation (#2729)
a36a997 
Referenced to: #2728

(cherry picked from commit a4dae9c)
mergify bot pushed a commit that referenced this issue Sep 15, 2022
@moriglia @mergify
Change description for SInt unary negation (#2729)
5f847ca 
Referenced to: #2728

(cherry picked from commit a4dae9c)
mergify bot added a commit that referenced this issue Sep 15, 2022
@mergify @moriglia
Change description for SInt unary negation (#2729) (#2734)
03f62c8 
Referenced to: #2728

(cherry picked from commit a4dae9c)

Co-authored-by: Marco Origlia <30799310+moriglia@users.noreply.github.com>
mergify bot added a commit that referenced this issue Sep 20, 2022
@mergify @moriglia
Change description for SInt unary negation (#2729) (#2733)
b961900 
Referenced to: #2728

(cherry picked from commit a4dae9c)

Co-authored-by: Marco Origlia <30799310+moriglia@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change SInt unary negation (expanding width) note
1 participant
@moriglia