-
Notifications
You must be signed in to change notification settings - Fork 269
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
Split ieee_floatt
into ieee_float_valuet
and ieee_floatt
#8550
base: develop
Are you sure you want to change the base?
Conversation
2e21587
to
4e6923f
Compare
ieee_floatt
into ieee_float_valuet
and ieee_floatt
4e6923f
to
076e2de
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #8550 +/- ##
===========================================
+ Coverage 78.79% 78.95% +0.15%
===========================================
Files 1730 1730
Lines 199160 198774 -386
Branches 18320 18376 +56
===========================================
+ Hits 156938 156950 +12
+ Misses 42222 41824 -398 ☔ View full report in Codecov by Sentry. |
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.
I very much like the idea, but I'm a bit confused: several places (some of which I've commented on below) appear to newly be drawing a rounding mode out of thin air when I'd naively have expected for those to be ieee_float_valuet
. Can you please clarify?
076e2de
to
8446d05
Compare
8446d05
to
5c8766d
Compare
4112d43
to
ac51457
Compare
This splits ieee_floatt into two parts: 1) ieee_float_valuet stores an IEEE 754 floating-point value. It offers no operations that require rounding, and hence, does not require a rounding mode. 2) ieee_floatt extends ieee_float_valuet with a rounding mode, and hence, offers operators that require rounding.
ac51457
to
be78e50
Compare
This splits
ieee_floatt
into two parts:ieee_float_valuet
stores an IEEE 754 floating-point value. It offers no operations that require rounding, and hence, does not require a rounding mode.ieee_floatt extends
ieee_float_valuet
with a rounding mode, and hence, offers operators that require rounding.