[libcxx-commits] [libcxx] [libc++][string] Add regression test for sized new/delete bug (PR #110210)
Vitaly Buka via libcxx-commits
libcxx-commits at lists.llvm.org
Thu Sep 26 23:09:42 PDT 2024
https://github.com/vitalybuka created https://github.com/llvm/llvm-project/pull/110210
This is regression test for #90292.
Allocator used in test is very similar to
test_allocator. However reproducer requires
size_type of the string to be 64bit, and
test_allocator has 32bit.
Fixes #92128.
>From 44da8d612670f97633a5c2832f2ef037c508af9b Mon Sep 17 00:00:00 2001
From: Vitaly Buka <vitalybuka at google.com>
Date: Thu, 26 Sep 2024 23:09:25 -0700
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=
=?UTF-8?q?l=20version?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Created using spr 1.3.4
---
.../string.capacity/deallocate_size.pass.cpp | 63 +++++++++++++++++++
1 file changed, 63 insertions(+)
create mode 100644 libcxx/test/std/strings/basic.string/string.capacity/deallocate_size.pass.cpp
diff --git a/libcxx/test/std/strings/basic.string/string.capacity/deallocate_size.pass.cpp b/libcxx/test/std/strings/basic.string/string.capacity/deallocate_size.pass.cpp
new file mode 100644
index 00000000000000..612b66195141c5
--- /dev/null
+++ b/libcxx/test/std/strings/basic.string/string.capacity/deallocate_size.pass.cpp
@@ -0,0 +1,63 @@
+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+// <string>
+
+// allocate/deallocate must match.
+
+#include <string>
+#include <cassert>
+#include <type_traits>
+
+#include "test_macros.h"
+
+static int allocated_;
+
+template <class T, class Sz>
+struct test_alloc {
+ typedef Sz size_type;
+ typedef std::make_signed<Sz>::type difference_type;
+ typedef T value_type;
+ typedef value_type* pointer;
+ typedef const value_type* const_pointer;
+ typedef typename std::add_lvalue_reference<value_type>::type reference;
+ typedef typename std::add_lvalue_reference<const value_type>::type const_reference;
+
+ template <class U>
+ struct rebind {
+ typedef test_alloc<U, Sz> other;
+ };
+
+ TEST_CONSTEXPR_CXX14 pointer allocate(size_type n, const void* = nullptr) {
+ allocated_ += n;
+ return std::allocator<value_type>().allocate(n);
+ }
+
+ TEST_CONSTEXPR_CXX14 void deallocate(pointer p, size_type s) {
+ allocated_ -= s;
+ std::allocator<value_type>().deallocate(p, s);
+ }
+};
+
+template <class Sz>
+void test() {
+ for (int i = 1; i < 1000; ++i) {
+ using Str = std::basic_string<char, std::char_traits<char>, test_alloc<char, Sz> >;
+ {
+ Str s(i, 't');
+ assert(allocated_ == 0 || allocated_ >= i);
+ }
+ }
+ assert(allocated_ == 0);
+}
+
+int main(int, char**) {
+ test<uint32_t>();
+ test<uint64_t>();
+ test<size_t>();
+}
More information about the libcxx-commits
mailing list