en Copyright 2015 Java http://src.rcs.uwaterloo.ca:8080/source/history/llvm-project/lldb/test/API/commands/disassemble/basic/main.cpp#826997c46280351861be43522d4a022d8fdbc466 [lldb] Fix a regression introduced by D75730In a new Range class was introduced to simplify and the Disassembler APIand reduce duplication. It unintentionally broke theSBFrame::Disassemble functionality because it unconditionally convertsthe number of instructions to a Range{Limit::Instructions,num_instructions}. This is subtly different from the previous behavior,where now we're passing a Range and assume it's valid in the callee, theoriginal code would propagate num_instructions and the callee wouldcompare the value and decided between disassembling instructions orbytes.Unfortunately the existing tests was not particularly strict: disassembly = frame.Disassemble() self.assertNotEqual(len(disassembly), 0, "Disassembly was empty.")This would pass because without this patch we'd disassemble zeroinstructions, resulting in an error: (lldb) script print(lldb.frame.Disassemble()) error: error reading data from section __textDifferential revision: https://reviews.llvm.org/D89925 List of files: /llvm-project/lldb/test/API/commands/disassemble/basic/main.cpp Thu, 22 Oct 2020 15:32:05 +0000 Jonas Devlieghere <jonas@devlieghere.com> http://src.rcs.uwaterloo.ca:8080/source/history/llvm-project/lldb/test/API/commands/disassemble/basic/main.cpp#fdea9a4ec9b0d9585b8fe8a612686d9f44f40ddc [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 List of files: /llvm-project/lldb/test/API/commands/disassemble/basic/main.cpp Thu, 20 Feb 2020 07:13:52 +0000 Raphael Isemann <teemperor@gmail.com> http://src.rcs.uwaterloo.ca:8080/source/history/llvm-project/lldb/test/API/commands/disassemble/basic/main.cpp#99451b4453688a94c6014cac233d371ab4cc342d [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 List of files: /llvm-project/lldb/test/API/commands/disassemble/basic/main.cpp Tue, 11 Feb 2020 18:01:16 +0000 Jordan Rupprecht <rupprecht@google.com>