This is an archive of the discontinued LLVM Phabricator instance.

[lit, test-suite] Fix a reference to FileBasedTest that I missed earlier.
ClosedPublic

Authored by dlj on Jun 29 2017, 12:26 PM.

Details

Summary

The test-suite implements its own specialized lit format for running tests,
which was based on FileBasedTest. Since that base was removed, the format now
needs to use something that is still available.

The only functionality that comes from FileBasedTest is the test discovery,
which was moved into ShTest. For now, it seems to make the most sense simply to
use ShTest as the base for TestSuiteTest.

I think a better solution is to factor out the test discovery logic, although
that will require a bit more work (and has effects across projects).

Diff Detail

Repository
rL LLVM

Event Timeline

dlj created this revision.Jun 29 2017, 12:26 PM
This revision was automatically updated to reflect the committed changes.

What happens when the user is using the lit from "pip install lit" instead of the one coming with llvm? Alternative question: Who is able to update the pypy package?

dlj added a comment.Jun 29 2017, 1:47 PM

What happens when the user is using the lit from "pip install lit" instead of the one coming with llvm?

That should be fine. ShTest has the same semantics before and after, so importing the old or the new version of ShTest will work.

Now, if they were using an old version of test-suite and the new version of lit, the answer would be different.

In D34830#795804, @dlj wrote:

What happens when the user is using the lit from "pip install lit" instead of the one coming with llvm?

That should be fine. ShTest has the same semantics before and after, so importing the old or the new version of ShTest will work.

Now, if they were using an old version of test-suite and the new version of lit, the answer would be different.

Ok, that's fine. Thanks for the fix.

leandron edited edge metadata.Jun 30 2017, 8:29 AM

@dlj Thanks for the fix. It runs fine with ToT lit and ToT test-suite.