This is an archive of the discontinued LLVM Phabricator instance.

[jenkins] Update script to use cross-project lit test suite
ClosedPublic

Authored by mib on Jun 29 2021, 10:26 AM.

Details

Summary

To reflect changes announced in https://lists.llvm.org/pipermail/llvm-dev/2021-January/148048.html, this patch updates all occurrences of debuginfo-tests to cross-project-debuginfo-tests.

Signed-off-by: Med Ismail Bennani <medismail.bennani@gmail.com>

Diff Detail

Event Timeline

mib created this revision.Jun 29 2021, 10:26 AM
mib requested review of this revision.Jun 29 2021, 10:26 AM
aprantl accepted this revision.Jun 29 2021, 1:58 PM
This revision is now accepted and ready to land.Jun 29 2021, 1:58 PM
jhenderson accepted this revision.Jun 29 2021, 11:57 PM

Looks good, with the exception of my inline comment.

zorg/jenkins/build.py
619 ↗(On Diff #355289)

Either I made a typo somewhere, or my comments were unclear apparently. This should be one of:

check-cross-project

or

check-debuginfo

There is no need to change it from what was there before, if you wish to maintain the behaviour as it was. If you change it to check-cross-project, additional tests may start getting picked up as they are added that aren't specifically debug info tests.

mib updated this revision to Diff 355494.Jun 30 2021, 3:22 AM

Address @jhenderson comment.