|
 |
53dace |
From db8f53df0be1daeda3159c1413549ff40696c710 Mon Sep 17 00:00:00 2001
|
|
|
53dace |
From: David Malcolm <dmalcolm@redhat.com>
|
|
|
53dace |
Date: Thu, 2 Sep 2021 17:02:33 -0400
|
|
|
53dace |
Subject: [PATCH 16/17] Conditionalize test for hashtable bucket sizes on
|
|
|
53dace |
__LIBSTDCXX_SO_VERSION >= 11
|
|
|
53dace |
|
|
|
53dace |
These tests were added upstream 2020-01-20 as part of:
|
|
|
53dace |
libstdc++: Do not over-size hashtable buckets on range insertion
|
|
|
53dace |
https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=6dcf042368012e2d7ce1626ee5d378bf3ad0ccfc
|
|
|
53dace |
|
|
|
53dace |
but fail when run in DTS against a system libstdc++.so from an older GCC.
|
|
|
53dace |
|
|
|
53dace |
In particular, _M_insert_unique_node from the header is using the older
|
|
|
53dace |
implementation of
|
|
|
53dace |
std::__detail::_Prime_rehash_policy::_M_need_rehash
|
|
|
53dace |
from the dynamic library.
|
|
|
53dace |
|
|
|
53dace |
23: 0000000000000000 0 FUNC GLOBAL DEFAULT UNDEF std::__detail::_Prime_rehash_policy::_M_need_rehash(unsigned long, unsigned long, unsigned long) const@GLIBCXX_3.4.18 (5)
|
|
|
53dace |
412: 0000000000000000 0 FUNC GLOBAL DEFAULT UNDEF std::__detail::_Prime_rehash_policy::_M_need_rehash(unsigned long, unsigned long, unsigned long) const@@GLIBCXX_3.4.18
|
|
|
53dace |
---
|
|
|
53dace |
.../23_containers/unordered_set/cons/bucket_hint.cc | 10 ++++++++++
|
|
|
53dace |
.../23_containers/unordered_set/modifiers/insert.cc | 9 +++++++++
|
|
|
53dace |
2 files changed, 19 insertions(+)
|
|
|
53dace |
|
|
|
53dace |
diff --git a/libstdc++-v3/testsuite/23_containers/unordered_set/cons/bucket_hint.cc b/libstdc++-v3/testsuite/23_containers/unordered_set/cons/bucket_hint.cc
|
|
|
53dace |
index a3b014a3a..af231e54e 100644
|
|
|
53dace |
--- a/libstdc++-v3/testsuite/23_containers/unordered_set/cons/bucket_hint.cc
|
|
|
53dace |
+++ b/libstdc++-v3/testsuite/23_containers/unordered_set/cons/bucket_hint.cc
|
|
|
53dace |
@@ -29,7 +29,11 @@ void test01()
|
|
|
53dace |
a.reserve(2);
|
|
|
53dace |
|
|
|
53dace |
std::unordered_set<int> b({ 0, 1, 0, 1, 0, 1, 0, 1 }, a.bucket_count());
|
|
|
53dace |
+
|
|
|
53dace |
+ // Fixed upstream in GCC 11
|
|
|
53dace |
+#if __LIBSTDCXX_SO_VERSION >= 11
|
|
|
53dace |
VERIFY( b.bucket_count() == a.bucket_count() );
|
|
|
53dace |
+#endif
|
|
|
53dace |
}
|
|
|
53dace |
|
|
|
53dace |
void test02()
|
|
|
53dace |
@@ -40,7 +44,10 @@ void test02()
|
|
|
53dace |
std::vector<int> v { 0, 1, 0, 1, 0, 1, 0, 1, 0, 1 };
|
|
|
53dace |
|
|
|
53dace |
std::unordered_set<int> b(v.begin(), v.end(), a.bucket_count());
|
|
|
53dace |
+ // Fixed upstream in GCC 11
|
|
|
53dace |
+#if __LIBSTDCXX_SO_VERSION >= 11
|
|
|
53dace |
VERIFY( b.bucket_count() == a.bucket_count() );
|
|
|
53dace |
+#endif
|
|
|
53dace |
}
|
|
|
53dace |
|
|
|
53dace |
void test03()
|
|
|
53dace |
@@ -51,7 +58,10 @@ void test03()
|
|
|
53dace |
std::forward_list<int> fl { 0, 1, 0, 1, 0, 1, 0, 1, 0, 1 };
|
|
|
53dace |
|
|
|
53dace |
std::unordered_set<int> b(fl.begin(), fl.end(), a.bucket_count());
|
|
|
53dace |
+ // Fixed upstream in GCC 11
|
|
|
53dace |
+#if __LIBSTDCXX_SO_VERSION >= 11
|
|
|
53dace |
VERIFY( b.bucket_count() == a.bucket_count() );
|
|
|
53dace |
+#endif
|
|
|
53dace |
}
|
|
|
53dace |
|
|
|
53dace |
int main()
|
|
|
53dace |
diff --git a/libstdc++-v3/testsuite/23_containers/unordered_set/modifiers/insert.cc b/libstdc++-v3/testsuite/23_containers/unordered_set/modifiers/insert.cc
|
|
|
53dace |
index 015c2f872..aae8298ae 100644
|
|
|
53dace |
--- a/libstdc++-v3/testsuite/23_containers/unordered_set/modifiers/insert.cc
|
|
|
53dace |
+++ b/libstdc++-v3/testsuite/23_containers/unordered_set/modifiers/insert.cc
|
|
|
53dace |
@@ -30,7 +30,10 @@ void test01()
|
|
|
53dace |
|
|
|
53dace |
auto bkt_count = a.bucket_count();
|
|
|
53dace |
a.insert({ 0, 1, 0, 1, 0, 1, 0, 1 });
|
|
|
53dace |
+ // Fixed upstream in GCC 11
|
|
|
53dace |
+#if __LIBSTDCXX_SO_VERSION >= 11
|
|
|
53dace |
VERIFY( a.bucket_count() == bkt_count );
|
|
|
53dace |
+#endif
|
|
|
53dace |
}
|
|
|
53dace |
|
|
|
53dace |
void test02()
|
|
|
53dace |
@@ -42,7 +45,10 @@ void test02()
|
|
|
53dace |
|
|
|
53dace |
auto bkt_count = a.bucket_count();
|
|
|
53dace |
a.insert(v.begin(), v.end());
|
|
|
53dace |
+ // Fixed upstream in GCC 11
|
|
|
53dace |
+#if __LIBSTDCXX_SO_VERSION >= 11
|
|
|
53dace |
VERIFY( a.bucket_count() == bkt_count );
|
|
|
53dace |
+#endif
|
|
|
53dace |
}
|
|
|
53dace |
|
|
|
53dace |
void test03()
|
|
|
53dace |
@@ -54,7 +60,10 @@ void test03()
|
|
|
53dace |
|
|
|
53dace |
auto bkt_count = a.bucket_count();
|
|
|
53dace |
a.insert(fl.begin(), fl.end());
|
|
|
53dace |
+ // Fixed upstream in GCC 11
|
|
|
53dace |
+#if __LIBSTDCXX_SO_VERSION >= 11
|
|
|
53dace |
VERIFY( a.bucket_count() == bkt_count );
|
|
|
53dace |
+#endif
|
|
|
53dace |
}
|
|
|
53dace |
|
|
|
53dace |
int main()
|
|
|
53dace |
--
|
|
|
53dace |
2.31.1
|
|
|
53dace |
|