[clang] Introduce paged vector (PR #66430)
Vassil Vassilev via cfe-commits
cfe-commits at lists.llvm.org
Sat Sep 30 12:49:20 PDT 2023
vgvassilev wrote:
@kuhar, thanks, you overtook me. Here is my diff:
```diff
diff --git a/llvm/unittests/ADT/PagedVectorTest.cpp b/llvm/unittests/ADT/PagedVectorTest.cpp
index e1b0c62d3395..c45df6d9e7e8 100644
--- a/llvm/unittests/ADT/PagedVectorTest.cpp
+++ b/llvm/unittests/ADT/PagedVectorTest.cpp
@@ -24,8 +24,8 @@ TEST(PagedVectorTest, EmptyTest) {
EXPECT_EQ(V.materialized_end().getIndex(), 0ULL);
EXPECT_EQ(std::distance(V.materialized_begin(), V.materialized_end()), 0LL);
- EXPECT_DEATH(V[0], "Index < Size");
- EXPECT_DEATH(PagedVector<int>(nullptr), "Allocator cannot be null");
+ EXPECT_DEBUG_DEATH(V[0], "Index < Size");
+ EXPECT_DEBUG_DEATH(PagedVector<int>(nullptr), "Allocator cannot be null");
}
TEST(PagedVectorTest, ExpandTest) {
@@ -69,7 +69,7 @@ TEST(PagedVectorTest, HalfPageFillingTest) {
for (int I = 0; I < 5; ++I)
EXPECT_EQ(V[I], I);
for (int I = 5; I < 10; ++I)
- EXPECT_DEATH(V[I], "Index < Size");
+ EXPECT_DEBUG_DEATH(V[I], "Index < Size");
}
TEST(PagedVectorTest, FillFullMultiPageTest) {
@@ -244,7 +244,7 @@ TEST(PagedVectorTest, ShrinkTest) {
EXPECT_EQ(V.size(), 0ULL);
EXPECT_EQ(V.capacity(), 0ULL);
EXPECT_EQ(std::distance(V.materialized_begin(), V.materialized_end()), 0LL);
- EXPECT_DEATH(V[0], "Index < Size");
+ EXPECT_DEBUG_DEATH(V[0], "Index < Size");
}
TEST(PagedVectorTest, FunctionalityTest) {
```
Could you use `EXPECT_DEBUG_DEATH` in your fix?
https://github.com/llvm/llvm-project/pull/66430
More information about the cfe-commits
mailing list