[libcxx-commits] [PATCH] D60246: [libc++] Simple contains for associative containers

Joe Loser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Apr 4 07:23:10 PDT 2019


jloser added a comment.

Sorry, I did not notice https://reviews.llvm.org/D59344 was in flight. That one looks like a superset of these changes, though I think we should only provide the declaration/definitions of `contains` in the case of `#if _LIBCPP_STD_VER > 17`. I will post a comment to that effect in https://reviews.llvm.org/D59344.

Going to abandon these changes since it's effectively a duplicate.


Repository:
  rCXX libc++

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

https://reviews.llvm.org/D60246





More information about the libcxx-commits mailing list