[libcxx-commits] [libcxx] [libc++] Add input validation for set_intersection() in debug mode. (PR #101508)
Louis Dionne via libcxx-commits
libcxx-commits at lists.llvm.org
Fri Aug 30 07:39:03 PDT 2024
================
@@ -43,33 +43,31 @@
#include "test_iterators.h"
-namespace {
-
-// __debug_less will perform an additional comparison in an assertion
-static constexpr unsigned std_less_comparison_count_multiplier() noexcept {
-#if _LIBCPP_HARDENING_MODE == _LIBCPP_HARDENING_MODE_DEBUG
- return 2;
+// Debug mode provides no complexity guarantees, testing them would be a waste of effort.
+#ifdef _LIBCPP_HARDENING_MODE_DEBUG
----------------
ldionne wrote:
```suggestion
#if defined(_LIBCPP_HARDENING_MODE_DEBUG) && _LIBCPP_HARDENING_MODE_DEBUG
```
We still need to check for `if defined(_LIBCPP_HARDENING_MODE_DEBUG)` to avoid `-Wundef` when testing non-libc++ libraries.
https://github.com/llvm/llvm-project/pull/101508
More information about the libcxx-commits
mailing list