-
Notifications
You must be signed in to change notification settings - Fork 15.2k
[LLVM-Test] Fix a regression in test dependencies introduced by 0e14973 #161623
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
Conversation
ef38b64
to
1f8f19d
Compare
Try no 3, restore UnitTests as a dependency for |
1f8f19d
to
81393a6
Compare
Thanks @chapuni for identifying this issue. This means our CI builds did not run LLVM unit tests during this period :( |
I verified that of the 3 references to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I expect it should work. Thanks for working.
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/76/builds/13060 Here is the relevant piece of the build log for the reference
|
… (llvm#161623) Add `UnitTests` as an explicit dependency for `check-llvm` and `llvm-test-depends`. In llvm#161442, the intent was to remove `UnitTests` as a dependency for the individual per-directory `check-llvm-*` test suites created but not to drop it from `check-llvm` or `llvm-test-depends`. This missing dependency will cause LLVM unit tests to be not rebuilt and resulting in either `warning: test suite 'LLVM-Unit' contained no tests` or running stale running versions of the unit tests when running `check-llvm`.
Add
UnitTests
as an explicit dependency forcheck-llvm
andllvm-test-depends
. In #161442, the intent was to removeUnitTests
as a dependency for the individual per-directorycheck-llvm-*
test suites created but not to drop it fromcheck-llvm
orllvm-test-depends
. This missing dependency will cause LLVM unit tests to be not rebuilt and resulting in eitherwarning: test suite 'LLVM-Unit' contained no tests
or running stale running versions of the unit tests when runningcheck-llvm
.