[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