[lldb] NFC: Fix trivial typo in comments, documents, and messagesDifferential Revision: https://reviews.llvm.org/D77460
[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
show more ...
[lldb][NFC] Remove 'from __future__ import print_function' from all tests that don't actually call 'print()'Summary:A lot of tests do this trick but the vast majority of them don't even call `prin
[lldb][NFC] Remove 'from __future__ import print_function' from all tests that don't actually call 'print()'Summary:A lot of tests do this trick but the vast majority of them don't even call `print()`.Most of this patch was generated by a script that just looks at all the files and deletes the line if there is no `print (` or `print(` anywhere else in the file.I checked the remaining tests manually and deleted the import if we never call print (but instead do stuff like `expr print(...)` and similar false-positives).I also corrected the additional empty lines after the import in the files that I manually edited.Reviewers: JDevlieghere, labath, jfbReviewed By: labathSubscribers: dexonsmith, wuzish, nemanjai, kbarton, christof, arphaman, abidh, lldb-commitsTags: #lldbDifferential Revision: https://reviews.llvm.org/D71452
[lldb][NFC] Remove unused imports in python testsllvm-svn: 367663
Bring Doxygen comment syntax in sync with LLVM coding style.This changes '@' prefix to '\'.llvm-svn: 355841
Update the file headers across all of the LLVM projects in the monorepoto reflect the new license.We understand that people may be surprised that we're moving the headerentirely to discuss the ne
Update the file headers across all of the LLVM projects in the monorepoto reflect the new license.We understand that people may be surprised that we're moving the headerentirely to discuss the new license. We checked this carefully with theFoundation's lawyer and we believe this is the correct approach.Essentially, all code in the project is now made available by the LLVMproject under our new license, so you will see that the license headersinclude that license only. Some of our contributors have contributedcode under our old license, and accordingly, we have retained a copy ofour old license notice in the top-level files in each project andrepository.llvm-svn: 351636
NFC: Fix some comment typos.llvm-svn: 333364
added Linux support for test timeout samplingThis is the Linux counterpart to the sampling support I addedon the macOS side.This change also introduces zip-file compression if the size ofthe sa
added Linux support for test timeout samplingThis is the Linux counterpart to the sampling support I addedon the macOS side.This change also introduces zip-file compression if the size ofthe sample output is greater than 10 KB. The Linux side can bequite large and the textual content is averaging over a 10xcompression factor on tests that I force to time out. Whencompression takes place, the filename becomes: {session_dir}/{TestFilename.py}-{pid}.sample.zipThis support relies on the linux 'perf' tool. If it isn'tpresent, the behavior is to ignore pre-kill processing ofthe timed out test process.Note calling the perf tool under the timeout command appearsto nuke the profiled process. This was causing the timeoutkill logic to fail due to the process having disappeared.I modified the kill logic to catch the case of the processnot existing, and I have it ignore the kill request in thatcase. Any other exception is still raised.Reviewers: labathSubscribers: lldb-commitsDifferential Revision: https://reviews.llvm.org/D24890llvm-svn: 282436
add hook for calling platform-dependent pre-kill action on a timed out testdifferential review: https://reviews.llvm.org/D24850reviewers: clayborg, labathllvm-svn: 282258
*** This commit represents a complete reformatting of the LLDB source code*** to conform to clang-format’s LLVM style. This kind of mass change has*** two obvious implications:Firstly, merging t
*** This commit represents a complete reformatting of the LLDB source code*** to conform to clang-format’s LLVM style. This kind of mass change has*** two obvious implications:Firstly, merging this particular commit into a downstream fork may be a hugeeffort. Alternatively, it may be worth merging all changes up to this commit,performing the same reformatting operation locally, and then discarding themerge for this particular commit. The commands used to accomplish thisreformatting were as follows (with current working directory as the root ofthe repository): find . \( -iname "*.c" -or -iname "*.cpp" -or -iname "*.h" -or -iname "*.mm" \) -exec clang-format -i {} + find . -iname "*.py" -exec autopep8 --in-place --aggressive --aggressive {} + ;The version of clang-format used was 3.9.0, and autopep8 was 1.2.4.Secondly, “blame” style tools will generally point to this commit instead ofa meaningful prior commit. There are alternatives available that will attemptto look through this change and find the appropriate prior commit. YMMV.llvm-svn: 280751
Revert "[test] Report error when inferior test processes exit with a non-zero code"This reverts r275782.The problem with the commit is that it reports an additional "exit (1)" error for every fil
Revert "[test] Report error when inferior test processes exit with a non-zero code"This reverts r275782.The problem with the commit is that it reports an additional "exit (1)" error for every filecontaining a failing test, which is far more than I had intended to do. I'll need to come up witha more fine-grained way of achieving the result.llvm-svn: 275791
[test] Report error when inferior test processes exit with a non-zero codeSummary:We've run into this problem when the test errored out so early (because it could not connect tothe remote device)
[test] Report error when inferior test processes exit with a non-zero codeSummary:We've run into this problem when the test errored out so early (because it could not connect tothe remote device), that the code in D20193 did not catch the error. This resulted in the testsuite reporting success with 0 tests being run.This patch makes sure that any non-zero exit code from the inferior process gets reported as anerror. Basically I expand the concept of "exceptional exits", which was previously being used forsignals to cover these cases as well.Reviewers: tfiala, zturnerSubscribers: lldb-commitsDifferential Revision: https://reviews.llvm.org/D22404llvm-svn: 275782
test infra cleanup: convert test_runner lib into packageAlso does the following:* adopts PEP8 naming convention for OptionalWith class (now optional_with).* moves test_runner/lldb_utils.py to l
test infra cleanup: convert test_runner lib into packageAlso does the following:* adopts PEP8 naming convention for OptionalWith class (now optional_with).* moves test_runner/lldb_utils.py to lldbsuite/support/optional_with.py.* packages tests in a subpackage of test_runner per recommendations in http://the-hitchhikers-guide-to-packaging.readthedocs.org/en/latest/creation.htmlTests can be run from within pacakges/Python/lldbsuite/test via thiscommand: python -m unittest discover test_runnerThe primary cleanup this allows is avoiding the need to muck with thePYTHONPATH variable from within the source files. This also aids someof the static code checkers as they don't need to run code to determinethe proper python path.llvm-svn: 266710
wire timeouts and exceptional inferior process exits through the test event systemThe results formatter system is now fed timeouts and exceptional processexits (i.e. inferior dotest.py process tha
wire timeouts and exceptional inferior process exits through the test event systemThe results formatter system is now fed timeouts and exceptional processexits (i.e. inferior dotest.py process that exited by signal on POSIXsystems).If a timeout or exceptional exit happens while a test method is runningon the worker queue, the timeout or exceptional exit is charged andreported against that test method. Otherwise, if no test method wasrunning at the time of the timeout or exceptional exit, only the testfilename will be reported as the TIMEOUT or ERROR.Implements:https://llvm.org/bugs/show_bug.cgi?id=24830https://llvm.org/bugs/show_bug.cgi?id=25703In support of:https://llvm.org/bugs/show_bug.cgi?id=25450llvm-svn: 255097
flip on executable bit on test runner testsllvm-svn: 255025
Python 3 - Use universal_newlines=True in subprocess.Popen.This follows the spirit of a previous patch which did essentiallythe same thing. In Python 3, when you use Popen.communicate(),you get
Python 3 - Use universal_newlines=True in subprocess.Popen.This follows the spirit of a previous patch which did essentiallythe same thing. In Python 3, when you use Popen.communicate(),you get back a bytes object which cannot normally be treated asa string. We could decode this manually, but universal_newlines=Truedoes this automatically, and there's no disadvantage to doing soeven on Python 2. So just enable it always.llvm-svn: 252126
Handle keyword args on our patched Popen methods.Python 3 introduces the `timeout` keyword argument on Popen.wait().If our patched version doesn't support keyword arguments, then whenthe internal
Handle keyword args on our patched Popen methods.Python 3 introduces the `timeout` keyword argument on Popen.wait().If our patched version doesn't support keyword arguments, then whenthe internal Python implementation attempts to call wait() with thekeyword argument, things will explode.Such as my head, after I finally figured out what was happening.llvm-svn: 252092
Move lldb/test to lldb/packages/Python/lldbsuite/test.This is the conclusion of an effort to get LLDB's Python codestructured into a bona-fide Python package. This has a numberof benefits, but m
Move lldb/test to lldb/packages/Python/lldbsuite/test.This is the conclusion of an effort to get LLDB's Python codestructured into a bona-fide Python package. This has a numberof benefits, but most notably the ability to more easily sharePython code between different but related pieces of LLDB's Pythoninfrastructure (for example, `scripts` can now share code with`test`).llvm-svn: 251532