[llvm-testresults] [llvm-commits] [llvm] r119600 - /llvm/trunk/unittests/VMCore/ValueMapTest.cpp

dalej dalej at apple.com
Wed Nov 17 17:33:26 PST 2010


This change seems to have broken the buildbots, please fix:

/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp: In member function 'virtual void<unnamed>::ValueMapTest_OperationsWork_Test<gtest_TypeParam_>::TestBody()':
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:75: error: 'BitcastV' was not declared in this scope
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:75: error: template argument 1 is invalid
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:77: error: 'AddV' was not declared in this scope
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:84: error: 'BitcastV' was not declared in this scope
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:84: error: template argument 1 is invalid
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:91: error: 'AddV' was not declared in this scope
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:91: error: template argument 1 is invalid
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:97: error: 'AddV' was not declared in this scope
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:97: error: template argument 1 is invalid
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp: In member function 'virtual void<unnamed>::ValueMapTest_Iteration_Test<gtest_TypeParam_>::TestBody()':
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:134: error: 'BitcastV' was not declared in this scope
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:134: error: template argument 1 is invalid
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:137: error: 'AddV' was not declared in this scope
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:137: error: template argument 1 is invalid
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:156: error: 'BitcastV' was not declared in this scope
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:156: error: template argument 1 is invalid
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:158: error: 'AddV' was not declared in this scope
/Users/buildslave/zorg/buildbot/smooshlab/slave/llvm-gcc-x86_64-darwin10-cross-i686-linux/llvm.src/unittests/VMCore/ValueMapTest.cpp:158: error: template argument 1 is invalid

On Nov 17, 2010, at 4:59 PM, Francois Pichet wrote:

> Author: fpichet
> Date: Wed Nov 17 18:59:50 2010
> New Revision: 119600
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=119600&view=rev
> Log:
> Appease MSVC 2008: you can't use keyword this inside EXPECT_EQ().
> This is because of bug 331418 on Microsoft Connect.
> 
> Modified:
>    llvm/trunk/unittests/VMCore/ValueMapTest.cpp
> 
> Modified: llvm/trunk/unittests/VMCore/ValueMapTest.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/VMCore/ValueMapTest.cpp?rev=119600&r1=119599&r2=119600&view=diff
> ==============================================================================
> --- llvm/trunk/unittests/VMCore/ValueMapTest.cpp (original)
> +++ llvm/trunk/unittests/VMCore/ValueMapTest.cpp Wed Nov 17 18:59:50 2010
> @@ -72,29 +72,29 @@
>   typename ValueMap<TypeParam*, int>::iterator I =
>     VM.find(this->BitcastV.get());
>   ASSERT_TRUE(I != VM.end());
> -  EXPECT_EQ(this->BitcastV.get(), I->first);
> +  EXPECT_EQ(BitcastV.get(), I->first);
>   EXPECT_EQ(7, I->second);
> -  EXPECT_TRUE(VM.find(this->AddV.get()) == VM.end());
> +  EXPECT_TRUE(VM.find(AddV.get()) == VM.end());
> 
>   // Const find:
>   const ValueMap<TypeParam*, int> &CVM = VM;
>   typename ValueMap<TypeParam*, int>::const_iterator CI =
>     CVM.find(this->BitcastV.get());
>   ASSERT_TRUE(CI != CVM.end());
> -  EXPECT_EQ(this->BitcastV.get(), CI->first);
> +  EXPECT_EQ(BitcastV.get(), CI->first);
>   EXPECT_EQ(7, CI->second);
>   EXPECT_TRUE(CVM.find(this->AddV.get()) == CVM.end());
> 
>   // Insert:
>   std::pair<typename ValueMap<TypeParam*, int>::iterator, bool> InsertResult1 =
>     VM.insert(std::make_pair(this->AddV.get(), 3));
> -  EXPECT_EQ(this->AddV.get(), InsertResult1.first->first);
> +  EXPECT_EQ(/*->this*/AddV.get(), InsertResult1.first->first);
>   EXPECT_EQ(3, InsertResult1.first->second);
>   EXPECT_TRUE(InsertResult1.second);
>   EXPECT_EQ(true, VM.count(this->AddV.get()));
>   std::pair<typename ValueMap<TypeParam*, int>::iterator, bool> InsertResult2 =
>     VM.insert(std::make_pair(this->AddV.get(), 5));
> -  EXPECT_EQ(this->AddV.get(), InsertResult2.first->first);
> +  EXPECT_EQ(/*this->*/AddV.get(), InsertResult2.first->first);
>   EXPECT_EQ(3, InsertResult2.first->second);
>   EXPECT_FALSE(InsertResult2.second);
> 
> @@ -131,10 +131,10 @@
>     std::pair<TypeParam*, int> value = *I; (void)value;
>     CompileAssertHasType<TypeParam*>(I->first);
>     if (I->second == 2) {
> -      EXPECT_EQ(this->BitcastV.get(), I->first);
> +      EXPECT_EQ(/*this->*/BitcastV.get(), I->first);
>       I->second = 5;
>     } else if (I->second == 3) {
> -      EXPECT_EQ(this->AddV.get(), I->first);
> +      EXPECT_EQ(/*this->*/AddV.get(), I->first);
>       I->second = 6;
>     } else {
>       ADD_FAILURE() << "Iterated through an extra value.";
> @@ -153,9 +153,9 @@
>     std::pair<TypeParam*, int> value = *I;  (void)value;
>     CompileAssertHasType<TypeParam*>(I->first);
>     if (I->second == 5) {
> -      EXPECT_EQ(this->BitcastV.get(), I->first);
> +      EXPECT_EQ(/*this->*/BitcastV.get(), I->first);
>     } else if (I->second == 6) {
> -      EXPECT_EQ(this->AddV.get(), I->first);
> +      EXPECT_EQ(/*this->*/AddV.get(), I->first);
>     } else {
>       ADD_FAILURE() << "Iterated through an extra value.";
>     }
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-testresults/attachments/20101117/b97ed2ec/attachment.html>


More information about the llvm-testresults mailing list