[libcxx-commits] [PATCH] D121530: [libc++][format] Implement format-string.

[Victor Zverovich via Phabricator via libcxx-commits]

vitaut added inline comments.


================
Comment at: libcxx/benchmarks/formatter_float.bench.cpp:237
+    std::array<char, s> r;
+    std::ranges::copy(make_fmt(), r.begin());
+    return r;
----------------
nit: no need to call make_fmt twice. In fact you could probably merge make_fmt into this lambda.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121530