[libcxx-commits] [PATCH] D60368: Add bind_front function (P0356R5)

[Marek Kurdej via Phabricator via libcxx-commits]

curdeius added a comment.

@zoecarver, for constexpr invoke, the paper of interest is http://wg21.link/P01065. It concerns some other functions as well, notably not_fn.



================
Comment at: libcxx/include/functional:2981
 invoke_result_t<_Fn, _Args...>
-invoke(_Fn&& __f, _Args&&... __args)
+_LIBCPP_CONSTEXPR_AFTER_CXX17 invoke(_Fn&& __f, _Args&&... __args)
     noexcept(is_nothrow_invocable_v<_Fn, _Args...>)
----------------
zoecarver wrote:
> curdeius wrote:
> > I guess you should add tests for constexpr to https://github.com/llvm/llvm-project/blob/master/libcxx/test/std/utilities/function.objects/func.invoke/invoke.pass.cpp.
> Good point. I should probably make this a different patch. 
> Good point. I should probably make this a different patch. 




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60368