[libc-commits] [PATCH] D115408: [libc] Implement correctly rounded logf based on RLIBM library.

Christoph Lauter via Phabricator via libc-commits libc-commits at lists.llvm.org
Fri Dec 17 12:23:14 PST 2021


cqlauter added inline comments.


================
Comment at: libc/src/math/generic/logf.cpp:173
+  FPBits f(xbits.val);
+  f.bits &= ~0x0000'FFFF;
+
----------------
Overall, I think that there should be way more comments for math functions like this one. A line like the one I am highlighting is so hard to understand, even when you are working on that code. Imagine you open up the code in 5 years and need to reunderstand a line like this.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115408



More information about the libc-commits mailing list