[libc-commits] [PATCH] D80779: [libc] Add implementations of round and roundf.

[Siva Chandra via Phabricator via libc-commits]

sivachandra marked 2 inline comments as done.
sivachandra added inline comments.


================
Comment at: libc/utils/FPUtil/FloatOperations.h:236-242
+  if (exponent == -1) {
+    // Absolute value of x is greater than equal to 0.5 but less than 1.
+    if (isNeg)
+      return T(-1.0);
+    else
+      return T(1.0);
+  }
----------------
asteinhauser wrote:
> Is this necessary?
After line 250, we want `exponent` to be less than mantissa width, but greater than 0. So, we handle these two special cases of -1 and -2 separately.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80779/new/

https://reviews.llvm.org/D80779