-
Notifications
You must be signed in to change notification settings - Fork 12.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Drop DejaGNU. #5589
Comments
The public buildbot has moved to using 'lit' for all tests. This decreased the cycle time by ~30% on the fast x86_64 builder. An additional step to do: |
Bug 5241 would be nice to have, but doesn't block dropping DejaGNU IMHO. |
#2 and #3 have been resolved. |
We should complete this for 2.7. |
2.7 only has a week or two to go... |
*** Bug llvm/llvm-bugzilla-archive#6391 has been marked as a duplicate of this bug. *** |
Made lit default for make check in r109997. |
#4 done in r110000, there is now an 'llvm-lit' tool which lives in the tool directory, and knows how to find the LLVM object root. |
#5 done in r110002. Last step is to rewrite TestingGuide.html. |
Updated testing guide basics in r110005. This more or less finishes off this bug -- there is lots of cleanup we can do to finish purging DejaGNU, but we don't need a bug for it. |
woohoo! |
r153408 and r153409 finalize purging DejaGNU from the sources of LLVM |
*** Bug llvm/llvm-bugzilla-archive#6390 has been marked as a duplicate of this bug. *** |
mentioned in issue llvm/llvm-bugzilla-archive#5240 |
mentioned in issue llvm/llvm-bugzilla-archive#6391 |
mentioned in issue llvm/llvm-bugzilla-archive#6390 |
[lldb] Fix issue with re.Pattern availability
Extended Description
The new 'lit" test runner seems to work well for LLVM and Clang. We should move towards phasing out DejaGNU:
The text was updated successfully, but these errors were encountered: