[libcxx-commits] [PATCH] D144275: [libc++][format] Addresses LWG3810.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante marked an inline comment as done.
Mordante added a subscriber: ldionne.
Mordante added inline comments.


================
Comment at: libcxx/include/__format/format_args.h:76
+template <class _Context, class... _Args>
+basic_format_args(__format_arg_store<_Context, _Args...>) -> basic_format_args<_Context>;
 
----------------
philnik wrote:
> Mordante wrote:
> > philnik wrote:
> > > Was there no test for the CTAD support before?
> > No that is the "silence the compiler when we want implicit CTAD" version. We don't have tests for these.
> I was asked by Louis to add one for the one I added for `move_iterator`(?). Weird that we don't have tests for all of them.
The original CTAD was added in D133535, but no test. I see some implicit CTAD tests added in that revision.
@ldionne should these test have been added for all types with the CTAD macro?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144275