r211625 - Disable the bits of r211623 that broke the bots

Eric Christopher echristo at gmail.com
Tue Jun 24 14:00:38 PDT 2014


On Tue, Jun 24, 2014 at 1:57 PM, Ben Langmuir <blangmuir at apple.com> wrote:
>
>> On Jun 24, 2014, at 1:51 PM, Eric Christopher <echristo at gmail.com> wrote:
>>
>> I missed that this is in a testcase and not in the main code. I'm
>> definitely less worked up about it, but perhaps some more detail than
>> just the FIXME would be nice :)
>
> Hey Eric,
>
> Fair enough, the comment sucks and you’re right this isn’t a great way to deal with test failures in general. FWIW, I’m working on this right now and I didn’t want to revert entirely, because I was hoping to get some reassurance from the bots that the rest of the patch was ok.  It’s hard to test filesystem-ish changes because of the potential for platform differences :-)
>

Yeah, you're absolutely right. I'd misread it and thought it was the
main code, but a testcase you just added that's parsing a bit weird on
a system you don't have I'm much less concerned about. Great that
you're looking at it right now, sometimes it's not clear if it's a
"now" versus "soon" thing. :)

Thanks!

-eric

> Cheers,
>
> Ben
>
>>
>> Thanks!
>>
>> -eric
>>
>> On Tue, Jun 24, 2014 at 1:39 PM, Eric Christopher <echristo at gmail.com> wrote:
>>> 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