r211625 - Disable the bits of r211623 that broke the bots

Eric Christopher echristo at gmail.com
Tue Jun 24 13:39:39 PDT 2014


Please don't do this in this way, just go ahead and revert the whole
patch and figure it out.

-eric

On Tue, Jun 24, 2014 at 1:00 PM, Ben Langmuir <blangmuir at apple.com> wrote:
> Author: benlangmuir
> Date: Tue Jun 24 15:00:30 2014
> New Revision: 211625
>
> URL: http://llvm.org/viewvc/llvm-project?rev=211625&view=rev
> Log:
> Disable the bits of r211623 that broke the bots
>
> Part of my test seems to rely on iterator bits that I didn't implement,
> at least in the gcc bots. Disabling while I investigate.
>
> Modified:
>     cfe/trunk/unittests/Basic/VirtualFileSystemTest.cpp
>
> Modified: cfe/trunk/unittests/Basic/VirtualFileSystemTest.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/Basic/VirtualFileSystemTest.cpp?rev=211625&r1=211624&r2=211625&view=diff
> ==============================================================================
> --- cfe/trunk/unittests/Basic/VirtualFileSystemTest.cpp (original)
> +++ cfe/trunk/unittests/Basic/VirtualFileSystemTest.cpp Tue Jun 24 15:00:30 2014
> @@ -395,22 +395,23 @@ TEST(VirtualFileSystemTest, HiddenInIter
>      checkContents(O->dir_begin("/", EC), Contents);
>    }
>
> +  // FIXME: broke gcc build
>    // Make sure we get the top-most entry
> -  vfs::directory_iterator E;
> -  {
> -    auto I = std::find_if(O->dir_begin("/", EC), E, [](vfs::Status S){
> -      return S.getName() == "/hiddenByUp";
> -    });
> -    ASSERT_NE(E, I);
> -    EXPECT_EQ(sys::fs::owner_all, I->getPermissions());
> -  }
> -  {
> -    auto I = std::find_if(O->dir_begin("/", EC), E, [](vfs::Status S){
> -      return S.getName() == "/hiddenByMid";
> -    });
> -    ASSERT_NE(E, I);
> -    EXPECT_EQ(sys::fs::owner_write, I->getPermissions());
> -  }
> +  // vfs::directory_iterator E;
> +  // {
> +  //   auto I = std::find_if(O->dir_begin("/", EC), E, [](vfs::Status S){
> +  //     return S.getName() == "/hiddenByUp";
> +  //   });
> +  //   ASSERT_NE(E, I);
> +  //   EXPECT_EQ(sys::fs::owner_all, I->getPermissions());
> +  // }
> +  // {
> +  //   auto I = std::find_if(O->dir_begin("/", EC), E, [](vfs::Status S){
> +  //     return S.getName() == "/hiddenByMid";
> +  //   });
> +  //   ASSERT_NE(E, I);
> +  //   EXPECT_EQ(sys::fs::owner_write, I->getPermissions());
> +  // }
>  }
>
>  // NOTE: in the tests below, we use '//root/' as our root directory, since it is
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits



More information about the cfe-commits mailing list