diff options
| author | Peter Maydell | 2014-01-06 12:47:21 +0100 |
|---|---|---|
| committer | Peter Maydell | 2014-01-08 20:07:22 +0100 |
| commit | 38970efafdfa8a992c177563c4f0d77062b88fd9 (patch) | |
| tree | 26655e0f499abbc1256856473823e0df70c7e2bd /include | |
| parent | hw: arm_gic: Introduce gic_set_priority function (diff) | |
| download | qemu-38970efafdfa8a992c177563c4f0d77062b88fd9.tar.gz qemu-38970efafdfa8a992c177563c4f0d77062b88fd9.tar.xz qemu-38970efafdfa8a992c177563c4f0d77062b88fd9.zip | |
softfloat: Fix exception flag handling for float32_to_float16()
Our float32 to float16 conversion routine was generating the correct
numerical answers, but not always setting the right set of exception
flags. Fix this, mostly by rearranging the code to more closely
resemble RoundAndPackFloat*, and in particular:
* non-IEEE halfprec always raises Invalid for input NaNs
* we need to check for the overflow case before underflow
* we weren't getting the tininess-detected-after-rounding
case correct (somewhat academic since only ARM uses halfprec
and it is always tininess-detected-before-rounding)
* non-IEEE halfprec overflow raises only Invalid, not
Invalid + Inexact
* we weren't setting Inexact when we should
Also add some clarifying comments about what the code is doing.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <rth@twiddle.net>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions