[PATCH] D88340: Patch IEEEFloat::isSignificandAllZeros and IEEEFloat::isSignificandAllOnes (bug 34579)
Andrew Briand via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 25 14:33:38 PDT 2020
andrewbriand created this revision.
andrewbriand added reviewers: gottesmm, chandlerc.
Herald added subscribers: dexonsmith, hiraditya.
Herald added a project: LLVM.
andrewbriand requested review of this revision.
Patch IEEEFloat::isSignificandAllZeros and IEEEFloat::isSignificandAllOnes to behave correctly in the case that the size of the significand is a multiple of the width of the integerParts making up the significand.
The patch to IEEEFloat::isSignificandAllOnes fixes bug 34579, and the patch to IEEE:Float:isSignificandAllZeros fixes the unit test "APFloatTest.x87Next" I added here. I have included both in this diff since the changes are very similar.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D88340
Files:
llvm/lib/Support/APFloat.cpp
llvm/unittests/ADT/APFloatTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88340.294420.patch
Type: text/x-patch
Size: 3666 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200925/902d51ea/attachment.bin>
More information about the llvm-commits
mailing list