Hello! I am looking for 1-2 volunteers to help moderate the lldb-dev and
lldb-commits mailing lists. Please let me know if you can help!
Thanks,
Tanya
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
ovyalov accepted this revision.
ovyalov added a comment.
LGTM
http://reviews.llvm.org/D12083
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
granata.enrico accepted this revision.
granata.enrico added a comment.
This revision is now accepted and ready to land.
Assuming it passes the test suite, LGTM
http://reviews.llvm.org/D12092
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
Author: jmolenda
Date: Mon Aug 17 19:21:24 2015
New Revision: 245261
URL: http://llvm.org/viewvc/llvm-project?rev=245261view=rev
Log:
Remove unintentional ;'s.
Modified:
lldb/trunk/source/Plugins/Process/Utility/DynamicRegisterInfo.cpp
Modified:
labath created this revision.
labath added reviewers: clayborg, ovyalov.
labath added subscribers: lldb-commits, dean.
in case we are logging to stdout, any log lines from the forked child can be
misconstrued to be
inferior output. To avoid this, we disable all logging immediately after
sdardis created this revision.
sdardis added a reviewer: clayborg.
sdardis added a subscriber: lldb-commits.
Change move encoding in PLTs and tests to be 'or' instead of 'addu'.
http://reviews.llvm.org/D12077
Files:
lib/ReaderWriter/ELF/Mips/MipsRelocationPass.cpp
clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.
Many changes. See inlined comments.
Comment at: source/Core/Disassembler.cpp:1169-1187
@@ +1168,21 @@
+
+/*
+ * MIPS:
+ * The bit #0 of
clayborg added a comment.
The main thing is, we don't want to be like other debuggers that have all this
code in many many places that check address bits by checking the Architecture
and litter the code with bit strips and adding bits where needed. We want to
support addresses correctly by
clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.
Looks good.
http://reviews.llvm.org/D12083
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
clayborg added a comment.
Please make sure that the following works after your changes:
(lldb) r
Process 35421 launched: '/private/tmp/a.out' (x86_64)
Process 35421 stopped
* thread #1: tid = 0xb659be, 0x00010f9d a.out main + 13, stop reason
= breakpoint 1.1, queue =
Merged to 3.7 in r245240. There was a conflicts because r245141 isn't
on the branch. Please check that I got it right:
http://llvm.org/viewvc/llvm-project?rev=245240view=rev
Thanks,
Hans
On Mon, Aug 17, 2015 at 1:28 PM, Greg Clayton clayb...@gmail.com wrote:
Ok to merge.
On Aug 17, 2015, at
clayborg resigned from this revision.
clayborg removed a reviewer: clayborg.
clayborg added a comment.
I don't think I am the right person to review this as this is in LLVM.
http://reviews.llvm.org/D12077
___
lldb-commits mailing list
I'm OK with it if Greg approves.
- Hans
On Mon, Aug 17, 2015 at 7:07 AM, sagar.tha...@imgtec.com wrote:
Hi Hans,
Could you please also merge r245217 in the release branch.
Thanks,
Sagar
-Original Message-
From: lldb-commits [mailto:lldb-commits-boun...@lists.llvm.org] On
+lldb-commits
On Mon, Aug 17, 2015 at 1:25 PM, Hans Wennborg h...@chromium.org wrote:
I'm OK with it if Greg approves.
- Hans
On Mon, Aug 17, 2015 at 7:06 AM, sagar.tha...@imgtec.com wrote:
Hi Hans,
Could you please merge r245216 in the release branch?
Thanks,
Sagar
-Original
14 matches
Mail list logo