[libcxx-commits] [PATCH] D109212: [libc++][NFCI] span: replace enable_if with requires

Joe Loser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Mar 14 18:24:25 PDT 2022


jloser added a comment.
Herald added a project: All.

@ldionne @Mordante now that all of our supported compilers support concepts, we can revisit this patch. What's your general appetite for using concepts (here and elsewhere for C++20-or-later code)? I prefer concepts, but I think Arthur had a benchmark showing `enable_if` was quicker to compile IIRC.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109212



More information about the libcxx-commits mailing list