[libcxx-commits] [PATCH] D114136: [libc++] Test that our algorithms never copy a user-provided comparator.

Nilay Vaish via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Nov 18 08:38:50 PST 2021


nilayvaish accepted this revision.
nilayvaish added inline comments.


================
Comment at: libcxx/include/__algorithm/is_heap_until.h:52
+_LIBCPP_NODISCARD_EXT inline _LIBCPP_HIDE_FROM_ABI _LIBCPP_CONSTEXPR_AFTER_CXX17 _RandomAccessIterator
+is_heap_until(_RandomAccessIterator __first, _RandomAccessIterator __last, _Compare __comp)
+{
----------------
nilayvaish wrote:
> Why does this need to be hidden?  Also isn't this already part of the ABI?
Found the ABI spec here: https://libcxxabi.llvm.org/spec.html.  So this seems good.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114136



More information about the libcxx-commits mailing list