[PATCH] D88340: Patch IEEEFloat::isSignificandAllZeros and IEEEFloat::isSignificandAllOnes (bug 34579)

[Andrew Briand via Phabricator via llvm-commits]

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>