[libcxx-commits] [PATCH] D93593: [libc++][format] Add __format_arg_store.

[Marek Kurdej via Phabricator via libcxx-commits]

curdeius added inline comments.


================
Comment at: libcxx/include/format:290
+// concepts is implemented. To ease the transition to using the not yet
+// implemetned concepts some helper concepts are added.
+// TODO FMT Switch to the real concepts when they are available.
----------------
Typo: implemented.


================
Comment at: libcxx/include/format:292
+// TODO FMT Switch to the real concepts when they are available.
+#if defined(__cpp_lib_concepts) && __cpp_lib_concepts >= 201806
+template <class _Tp>
----------------
We should make sure not to define __cpp_lib_concepts before having these concepts implemented then (or better, have all concepts implemented).
Personally, I'd rather just use below defined concepts and get rid of this ifdef.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93593