[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