[PATCH] D37128: [unittests] Limit reverse iteration test to only reverse iteration builds
Mandeep Singh Grang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 24 17:13:23 PDT 2017
mgrang created this revision.
This test causes failures in forward iteration builds. This is because we have hard-coded
the expected order of iteration of supported containers.
Repository:
rL LLVM
https://reviews.llvm.org/D37128
Files:
unittests/Support/ReverseIterationTest.cpp
Index: unittests/Support/ReverseIterationTest.cpp
===================================================================
--- unittests/Support/ReverseIterationTest.cpp
+++ unittests/Support/ReverseIterationTest.cpp
@@ -16,6 +16,8 @@
#include "llvm/Support/ReverseIteration.h"
#include "gtest/gtest.h"
+#if LLVM_ENABLE_REVERSE_ITERATION
+
using namespace llvm;
TEST(ReverseIterationTest, DenseMapTest1) {
@@ -109,3 +111,5 @@
for (auto iter = Set.begin(), end = Set.end(); iter != end; iter++, ++i)
ASSERT_EQ(*iter, IterPtrs[i]);
}
+
+#endif
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37128.112639.patch
Type: text/x-patch
Size: 554 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170825/50bb09cf/attachment.bin>
More information about the llvm-commits
mailing list