[libc-commits] [PATCH] D144161: [libc] Introduce a libc internal assertion macro named LIBC_ASSERT.

Tue Ly via Phabricator via libc-commits libc-commits at lists.llvm.org
Wed Feb 15 21:16:05 PST 2023


lntue added inline comments.


================
Comment at: libc/src/__support/float_to_string.h:67
+  LIBC_ASSERT(e >= 0 && e <= 1650 &&
+              "Incorrect exponent to perform log10_pow2 approximation.");
   return (e * 78913) >> 18;
----------------
I don't see anywhere in the `LIBC_ASSERT` definition that can take the error message.  Also, this is `&&` with the conditions in the previous line?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144161



More information about the libc-commits mailing list