Reverts llvm/llvm-project#157130 This was just a temporary change to investigate a flaky bot failure
Buildbot cross-project-tests-sie-ubuntu sees sporadic test failures due to missing "stackTrace" "source" "path". The "path" field is optional for "source" according to DAP, so it's well formed. It works most of the time, and doesn't consistently fail for any one test which is all strangely inconsistent. The failures couldn't be reproduced locally after running the feature_tests in a loop for 3 hours. I haven't been able to work out why the "source" is sometimes missing by just looking at LLDB code. This patch prints the DAP log to stderr when the specific error condition is hit, to help us understand why LLDB is sometimes getting into this state. Dump additional context from FileCheck so this can be seen on the bots (on a failure). This patch will be reverted once we've collected some logs.
This patch updates Dexter's lit test setup, making lldb-dap the default debugger where available; as part of this, I've replaced references to `--debugger lldb` with new substitutions that provide the debugger and executable path: one substitution returns just lldb for the tests that require it (all the LLDB-using tests in `dexter-tests`), and the other which provides the system-preferred debugger for the tests that should be able to run with any debugger (all the tests in `feature_tests`). As a small note, one test generates output that differs between lldb and lldb-dap - for this test specifically, we add an lldb-dap feature so that it can be marked XFAIL if we fall back to using LLDB.
This patch replaces invocations of clang with clang++ for a set of c++ files in the dexter cross-project tests. As a small additional change, this patch removes -lstdc++ from a test that did not appear to require it.
This patch makes a further attempt to fix the tests broken by the previous revision by ensuring that the command line for the modified Dexter tests use -std=gnu++11, in keeping with the old build script. This reverts commit 5647f2908de90fe07b0805e988cd2e91a1751928.
Re-application of the Dexter builder removal reversed due to continued errors on the green dragon LLDB buildbot: https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/59716/ Cause of the error is unclear, but it looks as though there is some unexpected non-determinism in the test failures. This reverts commit 323270451d8db24f2c816f455b3d8f70f434286d.
Fixes a test which was broken on the green dragon buildbot, and further failures on the SIE buildbot. This reverts commit 8df9eff90ff8c79e3e508cce7fec1cd897487069.
& Revert "[Dexter] Fix incorrect substitution errors in clang-cl builder" This reverts commits 262520a3c5450fd7f149438245b2aef12736347f, and 0b72b71cd3c8ad824342c05bc6d9d64d87eeb81b. Failures occurred on two buildbots, the SIE buildbot: https://lab.llvm.org/buildbot/#/builders/216/builds/26006 And the green dragon buildbot: https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/59091 Errors appear to be related to incorrect tool substitution in the Dexter test commands, and a currently unknown error with one of the general debuginfo tests that uses Dexter.
See "discussion": https://discourse.llvm.org/t/rfc-dexter-feature-removals/60462 This patch removes the builder functionality from Dexter, as it is an active maintenance burden and is no longer required since Dexter is being invoked by other test runners that can handle the build step better, and there has been no objection that it is still needed. Differential Revision: https://reviews.llvm.org/D151465
Discussion thread: https://lists.llvm.org/pipermail/llvm-dev/2021-January/148048.html Move debuginfo-test into a subdirectory of a new top-level directory, called cross-project-tests. The new name replaces "debuginfo-test" as an LLVM project enabled via LLVM_ENABLE_PROJECTS. Differential Revision: https://reviews.llvm.org/D95339 Reviewed by: aprantl