[lldb][tests] Automatically call compute_mydir (NFC)Eliminate boilerplate of having each test manually assign to `mydir` by calling`compute_mydir` in lldbtest.py.Differential Revision: https://r
[lldb][tests] Automatically call compute_mydir (NFC)Eliminate boilerplate of having each test manually assign to `mydir` by calling`compute_mydir` in lldbtest.py.Differential Revision: https://reviews.llvm.org/D128077
show more ...
[lldb] Remove anon struct from frame-var-anon-unions testThis test for anonymous unions seems off. It tests the following:```union { // fields};struct { // fields} var{...};```Both ar
[lldb] Remove anon struct from frame-var-anon-unions testThis test for anonymous unions seems off. It tests the following:```union { // fields};struct { // fields} var{...};```Both are anonymous types, but the first does not declare a variable and thesecond one does. The test then checks that `frame var` can directly access thefields of the anonymous union, but can't directly access the fields of theanonymous struct variable.The second test, to directly access the members of the struct variable, seemspointless as similar code would not compile. A demonstration:```struct { int a; int z;} a_z{23, 45};printf("%d\n", a_z.a); // fineprintf("%d\n", a); // this does not compile```Since we can't directly access the fields in code, I'm not sure there's areason to test that lldb also can't directly access them (other than perhaps asa regression test).Differential Revision: https://reviews.llvm.org/D116863
Revert "[lldb] Set result error state in 'frame variable'"This reverts commit 2bcff220bf1e372e91491911fe0bb76c4c4bbef8.
[lldb] Set result error state in 'frame variable'Ensure that errors in `frame variable` are reflected in result object.The statistics for `frame variable` show invocations as being successful, ev
[lldb] Set result error state in 'frame variable'Ensure that errors in `frame variable` are reflected in result object.The statistics for `frame variable` show invocations as being successful, evenwhen executing one of the error paths.This change replaces `result.GetErrorStream()` with `result.AppendError()`,which also sets the status to `eReturnStatusFailed`.Differential Revision: https://reviews.llvm.org/D116788(cherry picked from commit 2c7d10c41278181e3e45c68f28b501cd95193a8a)
[lldb] Remove license headers from all test source filesSummary:Around a third of our test sources have LLVM license headers. This patch removes those headers from all testsources and also fixes
[lldb] Remove license headers from all test source filesSummary:Around a third of our test sources have LLVM license headers. This patch removes those headers from all testsources and also fixes any tests that depended on the length of the license header.The reasons for this are:* A few tests verify line numbers and will start failing if the number of lines in the LLVM license header changes. Once I landed my patch for valid SourceLocations in debug info we will probably have even more tests that verify line numbers.* No other LLVM project is putting license headers in its test files to my knowledge.* They make the test sources much more verbose than they have to be. Several tests have longer license headers than the actual test source.For the record, the following tests had their line numbers changed to pass with the removal of the license header: lldb-api :: functionalities/breakpoint/breakpoint_by_line_and_column/TestBreakpointByLineAndColumn.py lldb-shell :: Reproducer/TestGDBRemoteRepro.test lldb-shell :: Reproducer/TestMultipleTargets.test lldb-shell :: Reproducer/TestReuseDirectory.test lldb-shell :: ExecControl/StopHook/stop-hook-threads.test lldb-shell :: ExecControl/StopHook/stop-hook.test lldb-api :: lang/objc/exceptions/TestObjCExceptions.pyReviewers: #lldb, espindola, JDevlieghereReviewed By: #lldb, JDevlieghereSubscribers: emaste, aprantl, arphaman, JDevlieghere, lldb-commitsTags: #lldbDifferential Revision: https://reviews.llvm.org/D74839
[lldb][test] Remove symlink for API tests.Summary: Moves lldbsuite tests to lldb/test/API.This is a largely mechanical change, moved with the following steps:```rm lldb/test/API/testcasesmkdi
[lldb][test] Remove symlink for API tests.Summary: Moves lldbsuite tests to lldb/test/API.This is a largely mechanical change, moved with the following steps:```rm lldb/test/API/testcasesmkdir -p lldb/test/API/{test_runner/test,tools/lldb-{server,vscode}}mv lldb/packages/Python/lldbsuite/test/test_runner/test lldb/test/API/test_runnerfor d in $(find lldb/packages/Python/lldbsuite/test/* -maxdepth 0 -type d | egrep -v "make|plugins|test_runner|tools"); do mv $d lldb/test/API; donefor d in $(find lldb/packages/Python/lldbsuite/test/tools/lldb-vscode -maxdepth 1 -mindepth 1 | grep -v ".py"); do mv $d lldb/test/API/tools/lldb-vscode; donefor d in $(find lldb/packages/Python/lldbsuite/test/tools/lldb-server -maxdepth 1 -mindepth 1 | egrep -v "gdbremote_testcase.py|lldbgdbserverutils.py|socket_packet_pump.py"); do mv $d lldb/test/API/tools/lldb-server; done```lldb/packages/Python/lldbsuite/__init__.py and lldb/test/API/lit.cfg.py were also updated with the new directory structure.Reviewers: labath, JDevlieghereTags: #lldbDifferential Revision: https://reviews.llvm.org/D71151