[llvm-commits] [llvm] r170369 - /llvm/trunk/unittests/Support/YAMLIOTest.cpp

Nick Kledzik kledzik at apple.com
Mon Dec 17 12:43:53 PST 2012


Author: kledzik
Date: Mon Dec 17 14:43:53 2012
New Revision: 170369

URL: http://llvm.org/viewvc/llvm-project?rev=170369&view=rev
Log:
re-enable test cases now that traits work with g++.  Fix some g++ warnings

Modified:
    llvm/trunk/unittests/Support/YAMLIOTest.cpp

Modified: llvm/trunk/unittests/Support/YAMLIOTest.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/Support/YAMLIOTest.cpp?rev=170369&r1=170368&r2=170369&view=diff
==============================================================================
--- llvm/trunk/unittests/Support/YAMLIOTest.cpp (original)
+++ llvm/trunk/unittests/Support/YAMLIOTest.cpp Mon Dec 17 14:43:53 2012
@@ -14,9 +14,6 @@
 #include "llvm/Support/YAMLTraits.h"
 #include "gtest/gtest.h"
 
-// To keep build bots going, disable tests until I figure out 
-// why gcc complains there is no match for these traits.
-#if 0
 
 using llvm::yaml::Input;
 using llvm::yaml::Output;
@@ -801,9 +798,9 @@
   EXPECT_FALSE(yin.error());
   EXPECT_EQ(seq.size(), 2UL);
   EXPECT_EQ(seq[0].kind,  kindA);
-  EXPECT_EQ(seq[0].flags, a2);
+  EXPECT_EQ(seq[0].flags, (uint32_t)a2);
   EXPECT_EQ(seq[1].kind,  kindB);
-  EXPECT_EQ(seq[1].flags, b1);
+  EXPECT_EQ(seq[1].flags, (uint32_t)b1);
 }
 
 //
@@ -831,15 +828,15 @@
     EXPECT_FALSE(yin.error());
     EXPECT_EQ(seq2.size(), 5UL);
     EXPECT_EQ(seq2[0].kind,  kindA);
-    EXPECT_EQ(seq2[0].flags, a1);
+    EXPECT_EQ(seq2[0].flags, (uint32_t)a1);
     EXPECT_EQ(seq2[1].kind,  kindB);
-    EXPECT_EQ(seq2[1].flags, b1);
+    EXPECT_EQ(seq2[1].flags, (uint32_t)b1);
     EXPECT_EQ(seq2[2].kind,  kindA);
-    EXPECT_EQ(seq2[2].flags, a2);
+    EXPECT_EQ(seq2[2].flags, (uint32_t)a2);
     EXPECT_EQ(seq2[3].kind,  kindB);
-    EXPECT_EQ(seq2[3].flags, b2);
+    EXPECT_EQ(seq2[3].flags, (uint32_t)b2);
     EXPECT_EQ(seq2[4].kind,  kindA);
-    EXPECT_EQ(seq2[4].flags, a3);
+    EXPECT_EQ(seq2[4].flags, (uint32_t)a3);
   }
 }
 
@@ -1288,5 +1285,3 @@
   EXPECT_TRUE(yin.error());
 }
 
-
-#endif





More information about the llvm-commits mailing list