[libc-commits] [PATCH] D152470: [libc][NFC] land long double table for printf

Michael Jones via Phabricator via libc-commits libc-commits at lists.llvm.org
Thu Jun 8 14:28:10 PDT 2023


michaelrj created this revision.
Herald added projects: libc-project, All.
Herald added a subscriber: libc-commits.
michaelrj requested review of this revision.

The Mega Table that printf uses for long doubles with some flags is too
large for the linters, and so has been split out from the main patch.
The main patch: https://reviews.llvm.org/D150399


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D152470

Files:
  libc/src/__support/ryu_long_double_constants.h
  libc/src/stdio/printf_core/CMakeLists.txt
  utils/bazel/llvm-project-overlay/libc/BUILD.bazel


Index: utils/bazel/llvm-project-overlay/libc/BUILD.bazel
===================================================================
--- utils/bazel/llvm-project-overlay/libc/BUILD.bazel
+++ utils/bazel/llvm-project-overlay/libc/BUILD.bazel
@@ -312,8 +312,7 @@
         "src/__support/ryu_constants.h",
         "src/__support/ryu_long_double_constants.h",
     ],
-    # This is temporarily commented out since the table is too large to land in the same patch as the rest of the changes.
-    # defines = ["LIBC_COPT_FLOAT_TO_STR_USE_MEGA_LONG_DOUBLE_TABLE"],
+    defines = ["LIBC_COPT_FLOAT_TO_STR_USE_MEGA_LONG_DOUBLE_TABLE"],
     deps = [
         ":__support_common",
         ":__support_cpp_type_traits",
Index: libc/src/stdio/printf_core/CMakeLists.txt
===================================================================
--- libc/src/stdio/printf_core/CMakeLists.txt
+++ libc/src/stdio/printf_core/CMakeLists.txt
@@ -99,8 +99,8 @@
     libc.src.__support.uint128
     libc.src.__support.integer_to_string
     libc.src.__support.float_to_string
-  # COMPILE_OPTIONS
-    # -DLIBC_COPT_FLOAT_TO_STR_USE_MEGA_LONG_DOUBLE_TABLE
+  COMPILE_OPTIONS
+    -DLIBC_COPT_FLOAT_TO_STR_USE_MEGA_LONG_DOUBLE_TABLE
 )
 
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D152470.529729.patch
Type: text/x-patch
Size: 1210 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libc-commits/attachments/20230608/5e067d53/attachment.bin>


More information about the libc-commits mailing list