[libcxx-commits] [PATCH] D116388: [libc++] Properly handle specializations of std::is_placeholder.

[Arthur O'Dwyer via Phabricator via libcxx-commits]

Quuxplusone updated this revision to Diff 397157.
Quuxplusone added a comment.

@ldionne: I believe your suggestion doesn't work (your `__is_placeholder<const _Tp>` specialization ends up unused, right?), but playing around with it got me to this version that I think works and is arguably cleaner — if much much metaprogrammier. :)  WDYT?


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

https://reviews.llvm.org/D116388

Files:
  libcxx/include/__functional/bind.h
  libcxx/test/std/utilities/function.objects/bind/func.bind/func.bind.isbind/is_bind_expression.pass.cpp
  libcxx/test/std/utilities/function.objects/bind/func.bind/func.bind.isbind/is_placeholder.pass.cpp
  libcxx/test/std/utilities/function.objects/bind/func.bind/func.bind.isbind/specialization.pass.cpp
  libcxx/test/std/utilities/function.objects/bind/func.bind/func.bind.isplace/is_placeholder.pass.cpp
  libcxx/test/std/utilities/function.objects/bind/func.bind/func.bind.isplace/specialization.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116388.397157.patch
Type: text/x-patch
Size: 7980 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220103/c027d229/attachment.bin>