[libc-commits] [PATCH] D131993: [libc] add compile option for printf arg type array

Siva Chandra via Phabricator via libc-commits libc-commits at lists.llvm.org
Thu Aug 25 14:59:57 PDT 2022


sivachandra accepted this revision.
sivachandra added inline comments.
This revision is now accepted and ready to land.


================
Comment at: libc/src/stdio/printf_core/printf_config.h:17
+// as the default size since that's big enough to handle even extreme cases and
+// the runtime penalty for not having enough space is severe.
+#ifndef LLVM_LIBC_PRINTF_INDEX_ARR_LEN
----------------
Nit: Give an example of what that "severity" will look like.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131993



More information about the libc-commits mailing list