[libc-commits] [PATCH] D150399: [libc] add options to printf decimal floats

Siva Chandra via Phabricator via libc-commits libc-commits at lists.llvm.org
Thu Jun 8 01:08:35 PDT 2023


sivachandra added inline comments.


================
Comment at: libc/src/__support/float_to_string.h:21
+
+#ifdef LIBC_COPT_FLOAT_TO_STR_USE_MEGA_TABLE
+#include "src/__support/ryu_long_double_constants.h"
----------------
michaelrj wrote:
> sivachandra wrote:
> > s/`LIBC_COPT_FLOAT_TO_STR_USE_MEGA_TABLE`/`LIBC_COPT_FLOAT_TO_STR_USE_LARGE_LONG_DOUBLE_TABLE`
> I like the name `MEGA_TABLE` since the table is several megabytes in size. Additionally, when the mega table is enabled it's used for both doubles and long doubles.
You can choose to keep `MEGA` in there but the name should reflect that it is about `long doubles`. So, I suggest to make it: `..._USE_MEGA_LONG_DOUBLE_TABLE`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150399



More information about the libc-commits mailing list