Skip to content

Commit 010a979

Browse files
committed
[ADT] Drop append_range test that doesn't compile with EXPENSIVE_CHECKS
The issue seems to be caused by the definition of `SafeIntIterator` and is otherwise unrelated to the implementation of `append_range`. We have a simialr test below, so this does not meaningfully decrease the test coverage of `append_range`. See the discussion in: https://reviews.llvm.org/D144420#4164373 Issue: #61122
1 parent 6956788 commit 010a979

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

llvm/unittests/ADT/STLExtrasTest.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
//===----------------------------------------------------------------------===//
88

99
#include "llvm/ADT/STLExtras.h"
10-
#include "llvm/ADT/Sequence.h"
1110
#include "llvm/ADT/StringRef.h"
1211
#include "gmock/gmock.h"
1312
#include "gtest/gtest.h"
@@ -405,9 +404,6 @@ TEST(STLExtrasTest, AppendRange) {
405404
append_range(V, AppendVals2);
406405
EXPECT_THAT(V, ElementsAre(1, 2, 3, 4, 5));
407406

408-
append_range(V, llvm::seq(6, 8));
409-
EXPECT_THAT(V, ElementsAre(1, 2, 3, 4, 5, 6, 7));
410-
411407
std::string Str;
412408
append_range(Str, "abc");
413409
EXPECT_THAT(Str, ElementsAre('a', 'b', 'c', '\0'));

0 commit comments

Comments
 (0)