From 269a931414ca88235c65f56c3b6970996fc4582c Mon Sep 17 00:00:00 2001 From: Daniel Bertalan Date: Tue, 15 Nov 2022 08:15:32 +0100 Subject: [PATCH] Tests/AK: Re-enable `HashTable` test The incorrect UBSan alignment check that made this test fail has been fixed in Clang 15. Closes #13614 --- Tests/AK/TestHashTable.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/Tests/AK/TestHashTable.cpp b/Tests/AK/TestHashTable.cpp index 57a7db0085..cccf4c879f 100644 --- a/Tests/AK/TestHashTable.cpp +++ b/Tests/AK/TestHashTable.cpp @@ -268,8 +268,6 @@ TEST_CASE(floats) EXPECT(table.contains(2.0f)); } -// FIXME: Enable this test once it doesn't trigger UBSAN. -#if 0 TEST_CASE(doubles) { HashTable table; @@ -281,7 +279,6 @@ TEST_CASE(doubles) EXPECT(table.contains(1.0)); EXPECT(table.contains(2.0)); } -#endif // Inserting and removing a bunch of elements will "thrash" the table, leading to a lot of "deleted" markers. BENCHMARK_CASE(benchmark_thrashing)