[Lldb-commits] [lldb] r372193 - TestFoundationDisassembly.py is not dependent on debug information.

Author: jingham Date: Tue Sep 17 17:40:49 2019 New Revision: 372193 URL: http://llvm.org/viewvc/llvm-project?rev=372193&view=rev Log: TestFoundationDisassembly.py is not dependent on debug information. This test is about disassembling symbols in a framework without debug information. So we don't

[Lldb-commits] [lldb] r372196 - Clean up this test.

Author: jingham Date: Tue Sep 17 18:53:52 2019 New Revision: 372196 URL: http://llvm.org/viewvc/llvm-project?rev=372196&view=rev Log: Clean up this test. I don't know what the intent of parts of this test were. We set a bunch of breakpoints and ran from one to the other, doing "self.runCmd("thre

[Lldb-commits] [lldb] r373135 - Give an error when StepUsingScriptedThreadPlan is passed a bad classname.

Author: jingham Date: Fri Sep 27 17:53:45 2019 New Revision: 373135 URL: http://llvm.org/viewvc/llvm-project?rev=373135&view=rev Log: Give an error when StepUsingScriptedThreadPlan is passed a bad classname. Differential Revision: https://reviews.llvm.org/D68173 Modified: lldb/trunk/include/

[Lldb-commits] [lldb] r373280 - Allow the internal-state-thread free access to the TargetAPI mutex.

Author: jingham Date: Mon Sep 30 17:47:25 2019 New Revision: 373280 URL: http://llvm.org/viewvc/llvm-project?rev=373280&view=rev Log: Allow the internal-state-thread free access to the TargetAPI mutex. It is always doing work on behalf of another thread that presumably has the mutex, so if it is

Re: [Lldb-commits] [PATCH] D68179: [lldb] Fix JSON parser to allow empty arrays

debugserver doesn't use llvm classes, it's its own stand-alone thing. So debugserver wasn't included in Jonas' changes. Jim > On Oct 1, 2019, at 3:11 PM, Alex Cameron via Phabricator > wrote: > > tetsuo-cpp added a comment. > > In D68179#1690073 , @

[Lldb-commits] [lldb] r373673 - Break out the Python class & key/value options into a separate OptionGroup.

Author: jingham Date: Thu Oct 3 15:18:51 2019 New Revision: 373673 URL: http://llvm.org/viewvc/llvm-project?rev=373673&view=rev Log: Break out the Python class & key/value options into a separate OptionGroup. Use this in the scripted breakpoint command. Added some tests for parsing the key/valu

[Lldb-commits] [lldb] r373675 - Pass an SBStructuredData to scripted ThreadPlans on use.

Author: jingham Date: Thu Oct 3 15:50:18 2019 New Revision: 373675 URL: http://llvm.org/viewvc/llvm-project?rev=373675&view=rev Log: Pass an SBStructuredData to scripted ThreadPlans on use. This will allow us to write reusable scripted ThreadPlans, since you can use key/value pairs with known ke

[Lldb-commits] [lldb] r373677 - Forgot to change the header guards on OptionGroupPythonClassWithDict.

Author: jingham Date: Thu Oct 3 16:32:42 2019 New Revision: 373677 URL: http://llvm.org/viewvc/llvm-project?rev=373677&view=rev Log: Forgot to change the header guards on OptionGroupPythonClassWithDict. I think that's what is confusing the modules build on the bots. Modified: lldb/trunk/inc

[Lldb-commits] [lldb] r373679 - Python3 doesn't seem to allow you to tell whether an object is a class

Author: jingham Date: Thu Oct 3 16:57:34 2019 New Revision: 373679 URL: http://llvm.org/viewvc/llvm-project?rev=373679&view=rev Log: Python3 doesn't seem to allow you to tell whether an object is a class PyClass_Check and everything it relied on seems gone from Python3.7. So I won't check wheth

[Lldb-commits] [lldb] r374226 - Revert "[lldb] Calculate relative path for symbol links"

Author: jingham Date: Wed Oct 9 13:56:43 2019 New Revision: 374226 URL: http://llvm.org/viewvc/llvm-project?rev=374226&view=rev Log: Revert "[lldb] Calculate relative path for symbol links" This reverts commit 958091c209d0a92e38b9cb27fb77a0ff7da11853. This commit incorrectly sets the _lldb.so s

Re: [Lldb-commits] [lldb] r296495 - Fix incorrect logic in StackFrame::Disassemble.

> On Feb 28, 2017, at 4:15 PM, Zachary Turner wrote: > > > > On Tue, Feb 28, 2017 at 3:49 PM Jim Ingham wrote: > > > On Feb 28, 2017, at 3:36 PM, Zachary Turner wrote: > > > > That patch was not really about early returns, it was about using > > StringRef. So my comment about the aestheti

[Lldb-commits] [lldb] r296669 - Add a sample_test directory with simple starter

Author: jingham Date: Wed Mar 1 14:25:48 2017 New Revision: 296669 URL: http://llvm.org/viewvc/llvm-project?rev=296669&view=rev Log: Add a sample_test directory with simple starter test cases for standard and "inline" tests. Added: lldb/trunk/packages/Python/lldbsuite/test/sample_test/ l

[Lldb-commits] [lldb] r296692 - Add a test to ensure that SBFrame::Disassemble produces some output.

Author: jingham Date: Wed Mar 1 16:18:37 2017 New Revision: 296692 URL: http://llvm.org/viewvc/llvm-project?rev=296692&view=rev Log: Add a test to ensure that SBFrame::Disassemble produces some output. Added: lldb/trunk/packages/Python/lldbsuite/test/functionalities/disassembly/TestFrameDis

[Lldb-commits] [lldb] r296693 - Make it clear what you should modify when you copy any of these sample

Author: jingham Date: Wed Mar 1 16:23:30 2017 New Revision: 296693 URL: http://llvm.org/viewvc/llvm-project?rev=296693&view=rev Log: Make it clear what you should modify when you copy any of these sample test cases. Modified: lldb/trunk/packages/Python/lldbsuite/test/sample_test/TestSampleI

[Lldb-commits] [lldb] r296814 - Added a list of outstanding projects that would benefit lldb.

Author: jingham Date: Thu Mar 2 15:39:27 2017 New Revision: 296814 URL: http://llvm.org/viewvc/llvm-project?rev=296814&view=rev Log: Added a list of outstanding projects that would benefit lldb. This was a list that I've had kicking around for a while, and would add to whenever some hallway con

[Lldb-commits] [lldb] r296816 - Goals->Projects.

Author: jingham Date: Thu Mar 2 15:42:00 2017 New Revision: 296816 URL: http://llvm.org/viewvc/llvm-project?rev=296816&view=rev Log: Goals->Projects. Modified: lldb/trunk/www/projects.html Modified: lldb/trunk/www/projects.html URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/www/projec

[Lldb-commits] [lldb] r296819 - Add a reference to the projects in the Get involved section.

Author: jingham Date: Thu Mar 2 15:45:39 2017 New Revision: 296819 URL: http://llvm.org/viewvc/llvm-project?rev=296819&view=rev Log: Add a reference to the projects in the Get involved section. Modified: lldb/trunk/www/index.html Modified: lldb/trunk/www/index.html URL: http://llvm.org/vie

[Lldb-commits] [lldb] r296826 - Forgot about local variable lookup.

Author: jingham Date: Thu Mar 2 16:04:05 2017 New Revision: 296826 URL: http://llvm.org/viewvc/llvm-project?rev=296826&view=rev Log: Forgot about local variable lookup. Yay for coffee lines. Modified: lldb/trunk/www/projects.html Modified: lldb/trunk/www/projects.html URL: http://llvm.o

[Lldb-commits] [lldb] r296833 - Mention fetching thread lists lazily.

Author: jingham Date: Thu Mar 2 16:13:45 2017 New Revision: 296833 URL: http://llvm.org/viewvc/llvm-project?rev=296833&view=rev Log: Mention fetching thread lists lazily. Modified: lldb/trunk/www/projects.html Modified: lldb/trunk/www/projects.html URL: http://llvm.org/viewvc/llvm-project/

[Lldb-commits] [lldb] r296834 - Python commands as first class citizens.

Author: jingham Date: Thu Mar 2 16:24:01 2017 New Revision: 296834 URL: http://llvm.org/viewvc/llvm-project?rev=296834&view=rev Log: Python commands as first class citizens. This should be a necessary precursor to adding support for any future extension languages. Modified: lldb/trunk/www/

Re: [Lldb-commits] [PATCH] D30560: Split DataExtractor into two classes.

That sounds right. Please do fix the function names. We can have another discussion about naming at some point, but we shouldn't do it piecemeal. Jim > On Mar 3, 2017, at 9:53 AM, Zachary Turner wrote: > > Yea, I can see splitting the target specific stuff into a separate "target > aware" d

Re: [Lldb-commits] [lldb] r296925 - Fix Darwin failures introduced in r296909

Yeah, looks like you need to keep DumpHexBytes in DumpDataExtractor. That's a little weird because it doesn't actually take a DataExtractor, but... Jim > On Mar 3, 2017, at 3:45 PM, Zachary Turner via lldb-commits > wrote: > > I can fix that, I didn't see it in my tests because it doesn't co

Re: [Lldb-commits] [lldb] r296925 - Fix Darwin failures introduced in r296909

I guess you could fudge by changing DumpDataExtractor.h to DataDumper.h to indicate that it is more general than just data extractors. Jim > On Mar 3, 2017, at 3:48 PM, Zachary Turner wrote: > > Yea, it was a static method of DataExtractor before. I noticed the same > thing when i saw it was

Re: [Lldb-commits] [lldb] r296925 - Fix Darwin failures introduced in r296909

I'll get this working, but in the future when you are making changes of this sort please chase down all the instances of functions you are changing, even in files you aren't building locally. There might be a use that your conversion doesn't handle, and you don't want to find that out after you'

Re: [Lldb-commits] [lldb] r296925 - Fix Darwin failures introduced in r296909

Yeah, you could have grepped (is that how you spell that?) for "#include "lldb/Core/DataExtractor.h" then looked at any Dump method in the files that turned up. That would have reduced the noise. Ah, staircase wit... Jim > On Mar 3, 2017, at 4:46 PM, Zachary Turner wrote: > > I usually do t

Re: [Lldb-commits] [lldb] r296925 - Fix Darwin failures introduced in r296909

Might also be useful to have a list of "files only compiled on platform X" somewhere. Then in this sort of case, you could compile, then scrutinize carefully all the files in the "not my platform list." Jim > On Mar 3, 2017, at 4:51 PM, Jim Ingham via lldb-commits &

[Lldb-commits] [lldb] r296938 - Fix the macOS build all the way after r296909.

Author: jingham Date: Fri Mar 3 19:15:24 2017 New Revision: 296938 URL: http://llvm.org/viewvc/llvm-project?rev=296938&view=rev Log: Fix the macOS build all the way after r296909. Modified: lldb/trunk/lldb.xcodeproj/project.pbxproj lldb/trunk/source/Plugins/Process/MacOSX-Kernel/Communic

Re: [Lldb-commits] [lldb] r296946 - Delete LLDB's code for getting / setting thread name.

This change seems to have lost the code that would make sure that on platforms with short thread names we pick up the end of the name passed in because that is generally the more specific part. Was that just an oversight? Jim > On Mar 3, 2017, at 5:31 PM, Zachary Turner via lldb-commits > wr

Re: [Lldb-commits] [lldb] r296946 - Delete LLDB's code for getting / setting thread name.

It was done on all platforms, but the way it worked before was there was a HostInfo::GetMaxThreadNameLength that was the length passed into SetThreadName, and then the string was back trimmed to that length generically in ThisThread::SetName. You can see the length passed in in HostNativeThrea

Re: [Lldb-commits] [PATCH] D30520: Make LLDB skip server-client roundtrip for signals that don't require any actions

> On Mar 6, 2017, at 4:10 PM, Greg Clayton via Phabricator > wrote: > > clayborg requested changes to this revision. > clayborg added a comment. > This revision now requires changes to proceed. > > Very close. Can we try to get UpdateAutomaticSignalFiltering out of > lldb_private::Process as

Re: [Lldb-commits] [PATCH] D30520: Make LLDB skip server-client roundtrip for signals that don't require any actions

The UnixSignals class produces the array of signal numbers that it knows it doesn't want to hear about. But it has no idea how any particular Process plugin would implement ignoring those symbols. So that part belongs to the Process plugin. I suggested in a previous comment also adding a Uni

Re: [Lldb-commits] [PATCH] D30520: Make LLDB skip server-client roundtrip for signals that don't require any actions

I like it in the base class and Greg was okay with that too. So let's leave that where it is. The only bit of this behavior that could be moved into UnixSignals as it seemed to me was the handling of "needs updating". I was mostly proposing that because then you can already pass the UnixSigna

Re: [Lldb-commits] [PATCH] D30520: Make LLDB skip server-client roundtrip for signals that don't require any actions

Yes formally that seems problematic. It wouldn't be a problem in practice because you should only call ResetNeedsUpdating in UpdateAutomaticSignalFiltering, and the only place where UpdateAutomaticSignalFiltering should be called is when Launching or Resuming, and you can't call either of thes

Re: [Lldb-commits] [lldb] r297830 - Fix TestMoveNearest breakage on darwin

Finding the executables that a binary will load on run before you actually run is always best effort. We don't require the ObjectFile class to fully emulate the platform loader. So from the standpoint of writing tests, if the test relies on getting symbols from a loaded library it should alway

[Lldb-commits] [lldb] r298289 - Fix a problem with line tables & .o files that start with code with no line table entries.

Author: jingham Date: Mon Mar 20 14:19:03 2017 New Revision: 298289 URL: http://llvm.org/viewvc/llvm-project?rev=298289&view=rev Log: Fix a problem with line tables & .o files that start with code with no line table entries. If you have code before the first line table entry when debugging with

[Lldb-commits] [lldb] r298290 - Get ObjectFileMachO to handle @executable_path

Author: jingham Date: Mon Mar 20 14:21:31 2017 New Revision: 298290 URL: http://llvm.org/viewvc/llvm-project?rev=298290&view=rev Log: Get ObjectFileMachO to handle @executable_path Only do this when we are debugging an executable, since we don't have a good way to trace from an ObjectFile back to

[Lldb-commits] [lldb] r298331 - FindTypes should find "struct TypeName" as well as "TypeName".

Author: jingham Date: Mon Mar 20 21:13:50 2017 New Revision: 298331 URL: http://llvm.org/viewvc/llvm-project?rev=298331&view=rev Log: FindTypes should find "struct TypeName" as well as "TypeName". This fixes a bug introduced by r291559. The Module's FindType was passing the original name not th

[Lldb-commits] [lldb] r298874 - Fix the Xcode project for OpenBSD additions.

Author: jingham Date: Mon Mar 27 14:03:11 2017 New Revision: 298874 URL: http://llvm.org/viewvc/llvm-project?rev=298874&view=rev Log: Fix the Xcode project for OpenBSD additions. Modified: lldb/trunk/lldb.xcodeproj/project.pbxproj Modified: lldb/trunk/lldb.xcodeproj/project.pbxproj URL: htt

[Lldb-commits] [lldb] r298876 - In FileSpec::Equal, short-cut GetNormalizedPath.

Author: jingham Date: Mon Mar 27 14:12:25 2017 New Revision: 298876 URL: http://llvm.org/viewvc/llvm-project?rev=298876&view=rev Log: In FileSpec::Equal, short-cut GetNormalizedPath. GetNormalizedPath seems to be slow, so it's worth shortcutting it if possible. This change does so when the filen

[Lldb-commits] [lldb] r298958 - Print the error if dsymForUUID sometimes produces bad plists.

Author: jingham Date: Tue Mar 28 18:25:34 2017 New Revision: 298958 URL: http://llvm.org/viewvc/llvm-project?rev=298958&view=rev Log: Print the error if dsymForUUID sometimes produces bad plists. Not much we can do about it but at least we can print the bad plist and the error. Modified: lld

Re: [Lldb-commits] [PATCH] D31371: Stop calling ValueObject::SetName from synthetic child providers

> On Mar 29, 2017, at 2:06 AM, Tamas Berghammer via Phabricator > wrote: > > tberghammer added a comment. > > SBValue::SetName is not part of the SB API (what is the right decision IMO as > an SBValue should be mostly immutable) so this issue doesn't effect it. I > looked through the code in

Re: [Lldb-commits] [PATCH] D31371: Stop calling ValueObject::SetName from synthetic child providers

I see. Might be worth filing an enhancement request to expose Clone so you can do this in a Python synthetic child provider. But there's no reason that lack should block this change. Jim > On Mar 30, 2017, at 12:51 PM, Tamas Berghammer wrote: > > It is possible to vend one of the actual bac

Re: [Lldb-commits] [PATCH] D31371: Stop calling ValueObject::SetName from synthetic child providers

Thanks. Jim > On Mar 30, 2017, at 1:16 PM, Tamas Berghammer wrote: > > Created bug for exposing ValueObject::Clone as SB API: > http://bugs.llvm.org/show_bug.cgi?id=32477 > > On Thu, Mar 30, 2017 at 1:04 PM Jim Ingham via Phabricator > wrote: > jingham accepted this revision. > jingham added

[Lldb-commits] [lldb] r299147 - Don't add a newline if the object description already has one.

Author: jingham Date: Thu Mar 30 20:32:57 2017 New Revision: 299147 URL: http://llvm.org/viewvc/llvm-project?rev=299147&view=rev Log: Don't add a newline if the object description already has one. Modified: lldb/trunk/source/DataFormatters/ValueObjectPrinter.cpp Modified: lldb/trunk/source

[Lldb-commits] [lldb] r299276 - DisassembleRange can return an empty DisassemblerSP

Author: jingham Date: Fri Mar 31 17:39:55 2017 New Revision: 299276 URL: http://llvm.org/viewvc/llvm-project?rev=299276&view=rev Log: DisassembleRange can return an empty DisassemblerSP check for it. Modified: lldb/trunk/source/Target/StackFrame.cpp Modified: lldb/trunk/source/Target/Stack

[Lldb-commits] [lldb] r299451 - Tone down the "lldb types" log a bit.

Author: jingham Date: Tue Apr 4 12:48:21 2017 New Revision: 299451 URL: http://llvm.org/viewvc/llvm-project?rev=299451&view=rev Log: Tone down the "lldb types" log a bit. Change the get shared class info function to only dump its results to the inferior stdout when the log is verbose. This matc

[Lldb-commits] [lldb] r299489 - Reverting r299374 & r299402 due to testsuite failure.

Author: jingham Date: Tue Apr 4 19:08:21 2017 New Revision: 299489 URL: http://llvm.org/viewvc/llvm-project?rev=299489&view=rev Log: Reverting r299374 & r299402 due to testsuite failure. This caused a failure in the test case: functionalities/breakpoint/objc/TestObjCBreakpoints.py When we ar

Re: [Lldb-commits] [PATCH] D31451: New C++ function name parsing logic

This patch is causing a testsuite failure in ObjC breakpoint setting: functionalities/breakpoint/objc/TestObjCBreakpoints.py You must be grabbing ObjC names and somehow treating them as C++ names. I reverted the patch till the test gets fixed. If you can't get your hands on an ObjC binary to t

Re: [Lldb-commits] [PATCH] D31451: New C++ function name parsing logic

No problem. You can't test on all platforms, so changes like this are bound to sometimes introduce regressions. If you can remember to watch the bots for systems you don't run or test on that makes the turnaround smoother. Otherwise, if it's a MacOS problem the QE folks here are sure to tell

[Lldb-commits] [lldb] r299609 - getAsInteger is not a equivalent replacement for strtol

Author: jingham Date: Wed Apr 5 20:33:38 2017 New Revision: 299609 URL: http://llvm.org/viewvc/llvm-project?rev=299609&view=rev Log: getAsInteger is not a equivalent replacement for strtol work around that fact for CommandObjectMemoryWrite. Modified: lldb/trunk/source/Commands/CommandObje

[Lldb-commits] [lldb] r300012 - Teach SBFrame how to guess its language.

Author: jingham Date: Tue Apr 11 19:19:54 2017 New Revision: 300012 URL: http://llvm.org/viewvc/llvm-project?rev=300012&view=rev Log: Teach SBFrame how to guess its language. Added: lldb/trunk/packages/Python/lldbsuite/test/functionalities/frame-language/ lldb/trunk/packages/Python/lld

Re: [Lldb-commits] [PATCH] D31969: [CMake] Support generating Config.h

> On Apr 11, 2017, at 7:01 PM, Zachary Turner via Phabricator via lldb-commits > wrote: > > zturner added a comment. > > How much would it complicate things to move the hand maintained files out of > tree? If the Xcode build isn't really a thing we're officially supporting, > perhaps we can

Re: [Lldb-commits] [PATCH] D31969: [CMake] Support generating Config.h

> On Apr 12, 2017, at 9:51 AM, Jim Ingham via lldb-commits > wrote: > >> >> On Apr 11, 2017, at 7:01 PM, Zachary Turner via Phabricator via lldb-commits >> wrote: >> >> zturner added a comment. >> >> How much would it complicate thing

[Lldb-commits] [lldb] r300517 - This test is succeeding on macOS with clang.

Author: jingham Date: Mon Apr 17 19:20:59 2017 New Revision: 300517 URL: http://llvm.org/viewvc/llvm-project?rev=300517&view=rev Log: This test is succeeding on macOS with clang. Modified: lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/class_static/TestStaticVariables.py Modified: lldb

[Lldb-commits] [lldb] r300519 - TestStaticVariables still fails on Linux.

Author: jingham Date: Mon Apr 17 19:44:14 2017 New Revision: 300519 URL: http://llvm.org/viewvc/llvm-project?rev=300519&view=rev Log: TestStaticVariables still fails on Linux. Modified: lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/class_static/TestStaticVariables.py Modified: lldb/tr

[Lldb-commits] [lldb] r300564 - Add back code to implement "frame var -a, -l, -g" filters.

Author: jingham Date: Tue Apr 18 11:52:16 2017 New Revision: 300564 URL: http://llvm.org/viewvc/llvm-project?rev=300564&view=rev Log: Add back code to implement "frame var -a,-l,-g" filters. r285226 dropped the code that did these checks. I am pretty sure that was inadvertent, so I added that ba

Re: [Lldb-commits] [PATCH] D32148: [Utility/StringLexer] Remove dead code.

We've had a couple of cases of "dead" code which turned out to be the llvm.org side of generic behavior for which Swift was the only current specific implementation. So while I agree, little single functions that aren't used anywhere are fine to cull, anything that looks like it might be the fo

[Lldb-commits] [lldb] r300733 - Add CopyDiagnostic to the DiagnosticManager.

Author: jingham Date: Wed Apr 19 13:56:44 2017 New Revision: 300733 URL: http://llvm.org/viewvc/llvm-project?rev=300733&view=rev Log: Add CopyDiagnostic to the DiagnosticManager. From Gregor Milos (gmi...@apple.com), for: https://reviews.llvm.org/D32078 Modified: lldb/trunk/include/lldb/Exp

[Lldb-commits] [lldb] r300785 - Fix !N and !-N commands and add a test case.

Author: jingham Date: Wed Apr 19 18:21:04 2017 New Revision: 300785 URL: http://llvm.org/viewvc/llvm-project?rev=300785&view=rev Log: Fix !N and !-N commands and add a test case. Added: lldb/trunk/packages/Python/lldbsuite/test/functionalities/history/ lldb/trunk/packages/Python/lldbsu

[Lldb-commits] [lldb] r300902 - Add an example command to toggle between disassembly-only and source mode.

Author: jingham Date: Thu Apr 20 16:51:27 2017 New Revision: 300902 URL: http://llvm.org/viewvc/llvm-project?rev=300902&view=rev Log: Add an example command to toggle between disassembly-only and source mode. Sometimes you are debugging in source, but you really only want to see the disassembly.

Re: [Lldb-commits] [lldb] r300902 - Add an example command to toggle between disassembly-only and source mode.

it more difficult to figure out how to do ordinary things. Jim > On Apr 21, 2017, at 1:32 AM, Pavel Labath wrote: > > This is cool, I just did some assembly debugging yesterday, and wished it was > easier :) > > On 20 April 2017 at 22:51, Jim Ingham via lldb-commits > wrote

[Lldb-commits] [lldb] r301608 - Add a newline to suppress compiler warnings.

Author: jingham Date: Thu Apr 27 19:44:07 2017 New Revision: 301608 URL: http://llvm.org/viewvc/llvm-project?rev=301608&view=rev Log: Add a newline to suppress compiler warnings. Modified: lldb/trunk/include/lldb/Core/TraceOptions.h Modified: lldb/trunk/include/lldb/Core/TraceOptions.h URL:

[Lldb-commits] [lldb] r301609 - Provide a mechanism to do some pre-loading of symbols up front.

Author: jingham Date: Thu Apr 27 19:51:06 2017 New Revision: 301609 URL: http://llvm.org/viewvc/llvm-project?rev=301609&view=rev Log: Provide a mechanism to do some pre-loading of symbols up front. Loading a shared library can require a large amount of work; rather than do that serially for each

Re: [Lldb-commits] [PATCH] D32832: Make ConstString creation and access lockfree

It isn't uncommon for folks to debug programs with all the debug information for the whole system available. In those cases, what looks to the user like a normal sized application is actually a very large one as far as lldb's global string pool is concerned. My experience is that whenever you

Re: [Lldb-commits] [PATCH] D32823: Remove an expensive lock from Timer

I'd vote for keeping the timers if possible. Their job is to tell you "of the time spent in some operation, how much was spent in say DWARF parsing", etc. That has been useful on occasion. Jim > On May 4, 2017, at 2:12 PM, Scott Smith via Phabricator via lldb-commits > wrote: > > scott.smi

[Lldb-commits] [lldb] r302323 - Added "info threads", "thread 1" and "apropos".

Author: jingham Date: Fri May 5 18:38:26 2017 New Revision: 302323 URL: http://llvm.org/viewvc/llvm-project?rev=302323&view=rev Log: Added "info threads", "thread 1" and "apropos". Modified: lldb/trunk/www/lldb-gdb.html Modified: lldb/trunk/www/lldb-gdb.html URL: http://llvm.org/viewvc/llv

[Lldb-commits] [lldb] r302327 - Be a little more permissive in DynamicLoaderMacOS::CanLoadImage

Author: jingham Date: Fri May 5 20:15:47 2017 New Revision: 302327 URL: http://llvm.org/viewvc/llvm-project?rev=302327&view=rev Log: Be a little more permissive in DynamicLoaderMacOS::CanLoadImage If we can't find the "is dyld locked" symbol, assume it is safe to load the image unless we only ha

Re: [Lldb-commits] [PATCH] D33283: RunThreadPlan: Fix halting logic in IgnoreBreakpoints = false

Yes, though the original code handled the unsuccessful completion cases slightly differently. I'm not sure how significant that is without pondering it further and I'm in the middle of something else right now. Jim > On May 17, 2017, at 1:31 PM, Pavel Labath via Phabricator > wrote: > > lab

[Lldb-commits] [lldb] r303643 - We shouldn't put actual tests in directories that contain

Author: jingham Date: Tue May 23 11:11:21 2017 New Revision: 303643 URL: http://llvm.org/viewvc/llvm-project?rev=303643&view=rev Log: We shouldn't put actual tests in directories that contain other test directories. Added: lldb/trunk/packages/Python/lldbsuite/test/functionalities/thread/num_

[Lldb-commits] [lldb] r303832 - Fix the warning when you pass -c to step/next/si/ni.

Author: jingham Date: Wed May 24 21:24:18 2017 New Revision: 303832 URL: http://llvm.org/viewvc/llvm-project?rev=303832&view=rev Log: Fix the warning when you pass -c to step/next/si/ni. During some cleanup the test for whether the thread plan accepted an iteration count was reversed, so we give

Re: [Lldb-commits] Documentation LLDB-MI

The virtue of the MI interface is that it allows you to write a tool that supports gdb as well as lldb. But the MI is a not an API per se, it's just a structured text interface. You send text commands to the MI server, and receive text results which you then parse to extract the results. If

Re: [Lldb-commits] [PATCH] D33426: Introduce new command: thread backtrace unique

SBThread::Resume instructs lldb to set the resume state on a thread to "eStateRunning", meaning that means the next time you continue the process, that thread will get a chance to run. It has no effect on the StopReason for the thread (it doesn't even start it running except maybe in the not we

Re: [Lldb-commits] [PATCH] D32930: New framework for lldb client-server communication tests.

Before we get past "it's hard" to "just to do it", it would help me to be clear first on what you are actually trying to achieve with this proposal. It's not clear to me what problem are people trying to solve here? If it is writing tests for the decomposable parts of lldb - like the tests Jas

Re: [Lldb-commits] [PATCH] D32930: New framework for lldb client-server communication tests.

ypothetical DSL that allowed every test to be an inline test but > still test equally complicated scenarios in half the code. Then suppose it > also ran on a more robust multiprocessing infrastructure than dotest.py. > That's all we're really talking about Thanks for the clarific

Re: [Lldb-commits] [PATCH] D32930: New framework for lldb client-server communication tests.

> devising. I can't at all see wanting to learn that if the only use it would > be to me is for writing tests. > > > After all, they're simpler than the other style of test. Now, suppose there > > were some hypothetical DSL that allowed every test to be an inli

[Lldb-commits] [lldb] r304379 - Forgot to mention rewriting CommandObject::DoExecute

Author: jingham Date: Wed May 31 20:05:30 2017 New Revision: 304379 URL: http://llvm.org/viewvc/llvm-project?rev=304379&view=rev Log: Forgot to mention rewriting CommandObject::DoExecute using the SB API's not the lldb_private API's. Modified: lldb/trunk/www/projects.html Modified: lldb/tru

Re: [Lldb-commits] [PATCH] D32022: Fix backtrace of noreturn functions situated at the end of a module

This is WWDC week. We’ll try to find time to take a look at this, but silence may mean preoccupation more than anything else… Jim > On Jun 6, 2017, at 6:53 AM, Pavel Labath via Phabricator > wrote: > > labath added a comment. > > I'd like to commit this this week. If you have any objections

[Lldb-commits] [lldb] r306445 - Fix up the Xcode project:

Author: jingham Date: Tue Jun 27 11:48:32 2017 New Revision: 306445 URL: http://llvm.org/viewvc/llvm-project?rev=306445&view=rev Log: Fix up the Xcode project: 1) Renaming the InstrumentationRuntime directory & file names 2) Bunch of stuff moved from Core to Utility 3) Deleted a bunch of files re

[Lldb-commits] [lldb] r306725 - Timer.{h, cpp} moved, find them again in the project file.

Author: jingham Date: Thu Jun 29 11:54:40 2017 New Revision: 306725 URL: http://llvm.org/viewvc/llvm-project?rev=306725&view=rev Log: Timer.{h,cpp} moved, find them again in the project file. Modified: lldb/trunk/lldb.xcodeproj/project.pbxproj Modified: lldb/trunk/lldb.xcodeproj/project.pbxp

[Lldb-commits] [lldb] r307234 - Add a lldbutils routine that gathers up the boiler-plate

Author: jingham Date: Wed Jul 5 19:18:16 2017 New Revision: 307234 URL: http://llvm.org/viewvc/llvm-project?rev=307234&view=rev Log: Add a lldbutils routine that gathers up the boiler-plate to make a target, set a source regex breakpoint, run to the breakpoint and find the thread that hit the br

[Lldb-commits] [lldb] r307287 - Working through testcases, converting to run_to_source_breakpoint.

Author: jingham Date: Thu Jul 6 11:06:25 2017 New Revision: 307287 URL: http://llvm.org/viewvc/llvm-project?rev=307287&view=rev Log: Working through testcases, converting to run_to_source_breakpoint. Modified: lldb/trunk/packages/Python/lldbsuite/test/expression_command/issue_11588/Test1158

[Lldb-commits] [lldb] r307870 - The llvm.org bugzilla moved.

Author: jingham Date: Wed Jul 12 17:36:21 2017 New Revision: 307870 URL: http://llvm.org/viewvc/llvm-project?rev=307870&view=rev Log: The llvm.org bugzilla moved. Modified: lldb/trunk/www/sidebar.incl Modified: lldb/trunk/www/sidebar.incl URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/

[Lldb-commits] [lldb] r307942 - Fix a deadlock in the Python interpreter vrs. SIGINT.

Author: jingham Date: Thu Jul 13 12:45:54 2017 New Revision: 307942 URL: http://llvm.org/viewvc/llvm-project?rev=307942&view=rev Log: Fix a deadlock in the Python interpreter vrs. SIGINT. The interpreter gets invoked in the sigint handler to cancel long-running Python operations. That requires t

[Lldb-commits] [lldb] r307943 - Convert a few more tests to use run_to_source_breakpoint.

Author: jingham Date: Thu Jul 13 12:46:21 2017 New Revision: 307943 URL: http://llvm.org/viewvc/llvm-project?rev=307943&view=rev Log: Convert a few more tests to use run_to_source_breakpoint. Modified: lldb/trunk/packages/Python/lldbsuite/test/expression_command/unwind_expression/TestUnwindE

[Lldb-commits] [lldb] r307944 - Enable parsing C++ names generated by lambda functions.

Author: jingham Date: Thu Jul 13 12:48:43 2017 New Revision: 307944 URL: http://llvm.org/viewvc/llvm-project?rev=307944&view=rev Log: Enable parsing C++ names generated by lambda functions. https://reviews.llvm.org/D34911 from Weng Xuetian. Modified: lldb/trunk/source/Plugins/Language/CPlusP

Re: [Lldb-commits] [PATCH] D34911: Enable parsing C++ names generated by lambda functions.

Done (r307944). Thanks for the patch! Jim > On Jul 13, 2017, at 8:20 AM, Xuetian Weng via Phabricator via lldb-commits > wrote: > > wengxt added a comment. > > I don't have commit access. May any one help me to commit? > > > https://reviews.llvm.org/D34911 > > > > _

[Lldb-commits] [lldb] r308549 - Add help text for "expression" telling how to enter multi-line mode.

Author: jingham Date: Wed Jul 19 16:25:42 2017 New Revision: 308549 URL: http://llvm.org/viewvc/llvm-project?rev=308549&view=rev Log: Add help text for "expression" telling how to enter multi-line mode. This seemed natural to us, but wasn't documented anywhere and was not clear to everybody. M

Re: [Lldb-commits] [PATCH] D35740: Fix PR33875 by distinguishing between DWO and clang modules

Not at present, but you presumably know more about this than I do. Part of the point of Greg's extracting the DWARF parser from lldb and making it into it's own library in llvm was precisely so that somebody could then write a simple wrapper tool that would poke it with not necessarily complete

[Lldb-commits] [lldb] r309238 - Fix the formatting for help on option value types.

Author: jingham Date: Wed Jul 26 17:18:18 2017 New Revision: 309238 URL: http://llvm.org/viewvc/llvm-project?rev=309238&view=rev Log: Fix the formatting for help on option value types. Patch by Jessica Han https://reviews.llvm.org/D35525 Modified: lldb/trunk/packages/Python/lldbsuite/test/

[Lldb-commits] [lldb] r309709 - Remember to make API headers Public in the LLDB target.

Author: jingham Date: Tue Aug 1 10:19:59 2017 New Revision: 309709 URL: http://llvm.org/viewvc/llvm-project?rev=309709&view=rev Log: Remember to make API headers Public in the LLDB target. Modified: lldb/trunk/lldb.xcodeproj/project.pbxproj Modified: lldb/trunk/lldb.xcodeproj/project.pbxpro

[Lldb-commits] [lldb] r309772 - Fix a mis-feature with propagation of breakpoint options -> location options.

Author: jingham Date: Tue Aug 1 17:16:10 2017 New Revision: 309772 URL: http://llvm.org/viewvc/llvm-project?rev=309772&view=rev Log: Fix a mis-feature with propagation of breakpoint options -> location options. When an option was set at on a location, I was just copying the whole option set to

[Lldb-commits] [lldb] r309969 - Add an auto-continue flag to breakpoints & locations.

Author: jingham Date: Thu Aug 3 11:13:24 2017 New Revision: 309969 URL: http://llvm.org/viewvc/llvm-project?rev=309969&view=rev Log: Add an auto-continue flag to breakpoints & locations. You can get a breakpoint to auto-continue by adding "continue" as a command, but that has the disadvantage th

[Lldb-commits] [lldb] r309977 - Cut and paste error from r23162.

Author: jingham Date: Thu Aug 3 12:38:38 2017 New Revision: 309977 URL: http://llvm.org/viewvc/llvm-project?rev=309977&view=rev Log: Cut and paste error from r23162. Modified: lldb/trunk/source/API/SBBreakpoint.cpp lldb/trunk/source/API/SBBreakpointLocation.cpp Modified: lldb/trunk/sour

Re: [Lldb-commits] [PATCH] D35223: Report inferior SIGSEGV/SIGILL/SIGBUS/SIGFPE as a signal instead of an exception on freebsd

> On Aug 8, 2017, at 12:02 AM, Pavel Labath via Phabricator via lldb-commits > wrote: > > labath added a comment. > > In https://reviews.llvm.org/D35223#834050, @emaste wrote: > >> With this patch I observed three new failures on FreeBSD and three new >> unexpected passes on FreeBSD. An exam

[Lldb-commits] [lldb] r311590 - Log whether we found a step out plan or not.

Author: jingham Date: Wed Aug 23 12:40:21 2017 New Revision: 311590 URL: http://llvm.org/viewvc/llvm-project?rev=311590&view=rev Log: Log whether we found a step out plan or not. Modified: lldb/trunk/source/Target/ThreadPlanStepInRange.cpp Modified: lldb/trunk/source/Target/ThreadPlanStepInR

[Lldb-commits] [lldb] r311679 - Fixed a typo in the example (getName -> GetName)

Author: jingham Date: Thu Aug 24 11:01:50 2017 New Revision: 311679 URL: http://llvm.org/viewvc/llvm-project?rev=311679&view=rev Log: Fixed a typo in the example (getName -> GetName) but while I was at it I converted the example to use properties, since that's much nicer looking. Modified: ll

[Lldb-commits] [lldb] r311786 - Add the DWARF DWP files to the Xcode project.

Author: jingham Date: Fri Aug 25 10:48:01 2017 New Revision: 311786 URL: http://llvm.org/viewvc/llvm-project?rev=311786&view=rev Log: Add the DWARF DWP files to the Xcode project. Modified: lldb/trunk/lldb.xcodeproj/project.pbxproj Modified: lldb/trunk/lldb.xcodeproj/project.pbxproj URL: ht

Re: [Lldb-commits] [PATCH] D37651: Fix for bug 34532 - A few rough corners related to post-mortem debugging (core/minidump)

Yes, the fact that llvm & clang and the MCJIT abort willy nilly has caused much pain to lldb. We don’t control that behavior and so are resigned to some continued suffering along those lines - though it seems like Lang is going to do a much cleaner job of handling problems in the new ORC JIT wh

Re: [Lldb-commits] [PATCH] D37651: Fix for bug 34532 - A few rough corners related to post-mortem debugging (core/minidump)

> On Sep 8, 2017, at 11:45 PM, Zachary Turner wrote: > > > On Fri, Sep 8, 2017 at 8:19 PM Jason Molenda > wrote: > Also keep in mind that debug sessions have a tendency to be long lived. I > may be working through a problem for half an hour -- or this may be the on

Re: [Lldb-commits] [PATCH] D37651: Fix for bug 34532 - A few rough corners related to post-mortem debugging (core/minidump)

> On Sep 9, 2017, at 11:36 AM, Zachary Turner wrote: > > > > On Sat, Sep 9, 2017 at 11:18 AM Jim Ingham > wrote: >> On Sep 8, 2017, at 11:45 PM, Zachary Turner > > wrote: >> > >> On Fri, Sep 8, 2017 at 8:19 PM Jason Molenda >

<    1   2   3   4   5   6   7   8   9   10   >