Author: jingham
Date: Wed Jul 19 16:25:42 2017
New Revision: 308549
URL: http://llvm.org/viewvc/llvm-project?rev=308549=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.
clayborg requested changes to this revision.
clayborg added a comment.
So we should just make sure this works:
(lldb) target symbols add --shlib a.out debug_binaries/a.out
The --uuid option is there to help you match up without having to specify the
file path. If I am understanding
Author: cbieneman
Date: Wed Jul 19 11:57:16 2017
New Revision: 308509
URL: http://llvm.org/viewvc/llvm-project?rev=308509=rev
Log:
Fix GreenDragon bots
This commit removes a very old deprecated API that was causing compile failures
for LLDB on Darwin. Since the comment says we only needed to
jingham added a comment.
This isn't about this patch, but replying to:
> debugserver replies to the k packet instead of just dropping the connection -
> stopping code adjusted, although we should probably consider aligning the
> behavior of the two stubs in this case
Especially when dealing
jingham requested changes to this revision.
jingham added a comment.
This revision now requires changes to proceed.
This seems like an okay change. You want to make sure that passing the file
name and UUID into AddModuleSymbols doesn't override a UUID mismatch against
the symbol file however
clayborg added a comment.
I am not sure how you expect to submit any patches then. Patches must be tested
prior to submission.
https://reviews.llvm.org/D33213
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
eugene added a comment.
In https://reviews.llvm.org/D35607#814560, @labath wrote:
> Btw, will this work correctly if we are cross-debugging from a mac? (because
> then we will use the fancy DownloadObjectAndSymbolFile implementation)
Yeah, you're right. I'll move this logic to the
Author: compnerd
Date: Wed Jul 19 08:46:21 2017
New Revision: 308489
URL: http://llvm.org/viewvc/llvm-project?rev=308489=rev
Log:
silence a couple of -Wqual-cast warning from GCC (NFC)
Cast to `const uint8_t *` instead of `uint8_t *` to avoid the warning
from GCC.
labath added a comment.
@beanz: It looks like you're doing some debugserver work. :) Any thoughts on
this?
https://reviews.llvm.org/D35311
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
labath added a comment.
Btw, will this work correctly if we are cross-debugging from a mac? (because
then we will use the fancy DownloadObjectAndSymbolFile implementation)
https://reviews.llvm.org/D35607
___
lldb-commits mailing list
This revision was automatically updated to reflect the committed changes.
Closed by commit rL308480: Expose hit count via SBBreakpointLocation. (authored
by brucem).
Repository:
rL LLVM
https://reviews.llvm.org/D31283
Files:
lldb/trunk/include/lldb/API/SBBreakpointLocation.h
Author: brucem
Date: Wed Jul 19 07:31:19 2017
New Revision: 308480
URL: http://llvm.org/viewvc/llvm-project?rev=308480=rev
Log:
Expose hit count via SBBreakpointLocation.
Summary:
SBBreakpointLocation exposed the ignore count, but didn't expose
the hit count. Both values were exposed by
brucem updated this revision to Diff 107297.
brucem added a comment.
Update per comments.
https://reviews.llvm.org/D31283
Files:
include/lldb/API/SBBreakpointLocation.h
packages/Python/lldbsuite/test/functionalities/breakpoint/breakpoint_hit_count/Makefile
krytarowski added a comment.
I don't have particular opinion. This change makes code cleaner.
https://reviews.llvm.org/D35618
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
labath created this revision.
The NativeThread class is useless without the containing process (and in
some places it is already assuming the process is always around). This
makes it clear that the NativeProcessProtocol is the object owning the
threads, and makes the destruction order
labath created this revision.
Herald added subscribers: mgorny, srhines.
Herald added a reviewer: EricWF.
https://reviews.llvm.org/D35615
Files:
packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/tuple/Makefile
labath accepted this revision.
labath added a comment.
The test looks well written. I've added a couple of suggestions you can
consider including.
Comment at:
packages/Python/lldbsuite/test/functionalities/breakpoint/breakpoint_hit_count/TestBreakpointHitCount.py:15
+
+
brucem added a reviewer: labath.
brucem added a comment.
Pavel, since Jim was okay with this before apart from it not having a test,
want to take a quick look now? Thanks!
https://reviews.llvm.org/D31283
___
lldb-commits mailing list
brucem updated this revision to Diff 107271.
brucem added a comment.
Add a test.
https://reviews.llvm.org/D31283
Files:
include/lldb/API/SBBreakpointLocation.h
packages/Python/lldbsuite/test/functionalities/breakpoint/breakpoint_hit_count/Makefile
This revision was automatically updated to reflect the committed changes.
Closed by commit rL308426: Fix typos in documentation. (authored by brucem).
Repository:
rL LLVM
https://reviews.llvm.org/D31282
Files:
lldb/trunk/scripts/interface/SBBreakpoint.i
Author: brucem
Date: Wed Jul 19 02:35:58 2017
New Revision: 308426
URL: http://llvm.org/viewvc/llvm-project?rev=308426=rev
Log:
Fix typos in documentation.
Reviewers: lldb-commits
Subscribers: lldb-commits
Differential Revision: https://reviews.llvm.org/D31282
Modified:
This revision was automatically updated to reflect the committed changes.
Closed by commit rL308425: Update API headers. (authored by brucem).
Repository:
rL LLVM
https://reviews.llvm.org/D35614
Files:
lldb/trunk/include/lldb/API/SBAttachInfo.h
lldb/trunk/include/lldb/API/SBBreakpoint.h
labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.
thanks
https://reviews.llvm.org/D35614
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
labath added a comment.
In https://reviews.llvm.org/D33213#814041, @DemiMarie wrote:
> In https://reviews.llvm.org/D33213#813037, @clayborg wrote:
>
> > Before any changes are submitted, we assume you are getting a clean test
> > suite run.
>
>
> On my system, I get build failures in clang.
I
24 matches
Mail list logo