-
Notifications
You must be signed in to change notification settings - Fork 38
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
Add Interval support #202
Add Interval support #202
Conversation
Derived IntervalTypeClass from FixedPointTypeClass - DspRealTypeClass needs asInterval - FixedPoint, SInt and UInt type classes needs asInterval - ChiselConvertableFrom get asInterval prototypes - ImplicitsTop gets Interval references - Fixed problem with poking negative fixed point and intervals into verilator backend
Tests working for this PR requires merge of PR #203 |
Dependent instead of PR #185 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Chick, this is great! Sorry to be so slow on the review
src/main/scala/dsptools/numbers/chisel_types/FixedPointTypeClass.scala
Outdated
Show resolved
Hide resolved
// Note: The above doesn't rely on tools to expand, shrink correctly; the version below does. | ||
// Assumes setBinaryPoint zero-extends. BUT Chisel doesn't seem to get widths properly and | ||
// some other ops rely on width correctness... (even though Firrtl is right...) | ||
//a.setBinaryPoint(newBP) >> n |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is unfortunate. do you think our goal should be to ultimately make setBinaryPoint work?
- fixed maskeValue typo - cleaned up big ugly import
Derived IntervalTypeClass from FixedPointTypeClass