[clang] [llvm] Enable LLDB tests in Linux pre-merge CI (PR #94208)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 7 11:11:13 PDT 2024


AaronBallman wrote:

I have no objections in theory, but in practice I remain concerned about the flakiness of lldb's tests for the impacts on Clang reviewers.

There are timeout failures, but they might be limited to Windows and thus not a problem with your PR (https://lab.llvm.org/buildbot/#/builders/219/builds/11819 and
https://lab.llvm.org/buildbot/#/builders/219/builds/11810 as examples).

But then there are failures like:
https://lab.llvm.org/buildbot/#/builders/17/builds/53939 -- this same stack trace seems to pop up in other failures and looks to be unrelated to changes in the change set, such as https://lab.llvm.org/buildbot/#/builders/68/builds/75367

or

https://lab.llvm.org/buildbot/#/builders/68/builds/75698 -- different assertion, but like above, seems to be spurious and unrelated to changes in the changeset, such as https://lab.llvm.org/buildbot/#/builders/68/builds/75666

Perhaps one way forward is for lldb tests to run on lldb changes for a few months before enabling them on Clang changes? Then the lldb folks can either report back that things are in pretty good shape from their experience, or we'll maybe have a better idea of how to proceed.

That said, if others aren't as concerned about test flakiness, I won't block progress on this PR.

https://github.com/llvm/llvm-project/pull/94208


More information about the cfe-commits mailing list