Re: [Lldb-commits] [PATCH] D14633: [LLDB][MIPS] Clear bug 25194 - LLDB-Server Assertion raised when single stepping on MIPS

2015-11-23 Thread Sagar Thakur via lldb-commits
sagar added a comment. Hi, Could we use SetBytes for now for clearing the bug 25194? I have tried using SetBytes(), it does not cause any issue on MIPS for both endian. Once we have a new function to llvm::APInt to access actual data I will revert back to using SetUInt. Kindly let me know if y

Re: [Lldb-commits] [PATCH] D14895: Prevent ProcessGDBRemote from launching local debug server in case of remote debug server connection failure

2015-11-23 Thread Pavel Labath via lldb-commits
labath accepted this revision. labath added a comment. This revision is now accepted and ready to land. Nice catch. Looks good. Comment at: source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp:3516 @@ -3544,1 +3515,3 @@ Error +ProcessGDBRemote::CheckDebugserverConnection (con

Re: [Lldb-commits] [PATCH] D14904: Document NetBSD platform support on the webpage

2015-11-23 Thread Pavel Labath via lldb-commits
labath added a subscriber: labath. Comment at: www/build.html:345 @@ -344,3 +352,1 @@ - LLDB has a Python scripting capability and supplies its own Python module named lldb. - If a script is run inside the command line lldb application, th

Re: [Lldb-commits] [PATCH] Define new builder_netbsd

2015-11-23 Thread Kamil Rytarowski via lldb-commits
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 For the reference, there was availability outage during this weekend on reviews.llvm.org. I was suggested to send a patch to this mailing list as a fall-back. Please review and merge with the LLDB sources. On 22.11.2015 17:23, Kamil Rytarowski via

Re: [Lldb-commits] [PATCH] D14823: Disable forcing -marm (A32 instruction set) while running testsuite on arm targets.

2015-11-23 Thread Muhammad Omair Javaid via lldb-commits
omjavaid added a comment. I agree with you on having multiple configuration options to help identify failures in different configurations. What I mean is that we should keep clarity on architecture that is to use arm or aarch32 for 32bit and aarch64 for 64bit. With that we should run tests in

Re: [Lldb-commits] [PATCH] D14860: [LLDB][MIPS] Getting 0 index for h/w watchpoint is not necessarily an error

2015-11-23 Thread Jaydeep Patil via lldb-commits
jaydeep accepted this revision. jaydeep added a comment. This revision is now accepted and ready to land. Looks good to me. Repository: rL LLVM http://reviews.llvm.org/D14860 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.

[Lldb-commits] [lldb] r253864 - [LLDB][MIPS] Getting 0 index for H/W watchpoint is not necessarily an error

2015-11-23 Thread Mohit K. Bhakkad via lldb-commits
Author: mohit.bhakkad Date: Mon Nov 23 06:19:59 2015 New Revision: 253864 URL: http://llvm.org/viewvc/llvm-project?rev=253864&view=rev Log: [LLDB][MIPS] Getting 0 index for H/W watchpoint is not necessarily an error Reviewers: jaydeep. Subscribers: bhushan, sagar, nitesh.jain, lldb-commits. Diffe

Re: [Lldb-commits] [PATCH] D14860: [LLDB][MIPS] Getting 0 index for h/w watchpoint is not necessarily an error

2015-11-23 Thread Mohit Bhakkad via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL253864: [LLDB][MIPS] Getting 0 index for H/W watchpoint is not necessarily an error (authored by mohit.bhakkad). Changed prior to commit: http://reviews.llvm.org/D14860?vs=40744&id=40913#toc Repository

Re: [Lldb-commits] [PATCH] D12877: [LLDB] Switch to assembly view if source is moved

2015-11-23 Thread Mohit Bhakkad via lldb-commits
mohit.bhakkad added a comment. @jingham, @jasonmolenda, is it okay to commit it? Repository: rL LLVM http://reviews.llvm.org/D12877 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-comm

Re: [Lldb-commits] [PATCH] D14823: Disable forcing -marm (A32 instruction set) while running testsuite on arm targets.

2015-11-23 Thread Tamas Berghammer via lldb-commits
tberghammer accepted this revision. tberghammer added a comment. I agree with your suggestion to make this the default config and then add a new option where we can specify additional target specific options and then skip/xfail tests based on that. http://reviews.llvm.org/D14823 ___

Re: [Lldb-commits] [PATCH] D14895: Prevent ProcessGDBRemote from launching local debug server in case of remote debug server connection failure

2015-11-23 Thread Oleksiy Vyalov via lldb-commits
ovyalov updated this revision to Diff 40948. ovyalov added a comment. Addressed review comments - renamed the method to EstablishConnectionIfNeeded. http://reviews.llvm.org/D14895 Files: source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp source/Plugins/Process/gdb-remote/ProcessGDBRemot

[Lldb-commits] LLVM buildmaster will be restarted tonight

2015-11-23 Thread Galina Kistanova via lldb-commits
Hello everyone, LLVM buildmaster will be updated restarted after 7 PM Pacific time today. Thanks Galina ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] r253906 - Prevent ProcessGDBRemote from launching local debug server in case of remote debug server connection failure.

2015-11-23 Thread Oleksiy Vyalov via lldb-commits
Author: ovyalov Date: Mon Nov 23 13:32:24 2015 New Revision: 253906 URL: http://llvm.org/viewvc/llvm-project?rev=253906&view=rev Log: Prevent ProcessGDBRemote from launching local debug server in case of remote debug server connection failure. http://reviews.llvm.org/D14895 Modified: lldb/

Re: [Lldb-commits] [PATCH] D14895: Prevent ProcessGDBRemote from launching local debug server in case of remote debug server connection failure

2015-11-23 Thread Oleksiy Vyalov via lldb-commits
ovyalov closed this revision. ovyalov added a comment. Files: /lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp /lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.h Users: ovyalov (Author) http://reviews.llvm.org/rL253906 http://reviews.llvm.org/D14895 _

Re: [Lldb-commits] [PATCH] D10247: Fix crash when cu_comp_dir parameter to DWARFDebugLine::ParseSupportFiles() is an empty string

2015-11-23 Thread Ted Woodward via lldb-commits
ted abandoned this revision. ted added a comment. Handled by another patch http://reviews.llvm.org/D10247 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D14932: Fix watchpoint check to use watchpoint ranges

2015-11-23 Thread Ted Woodward via lldb-commits
ted created this revision. ted added a reviewer: clayborg. ted added a subscriber: lldb-commits. Watchpoints, unlike breakpoints, have an address range. This patch changes WatchpointList::FindByAddress() to match on any address in the watchpoint range, instead of only matching on the watchpoint'

[Lldb-commits] [PATCH] D14937: swig_bot remote path preliminary implementation

2015-11-23 Thread Zachary Turner via lldb-commits
zturner created this revision. zturner added a reviewer: tfiala. zturner added a subscriber: lldb-commits. With this patch, the client will package up all the required inputs into a compressed zip file, establish a connection to the server, send the input to the server, and wait for th

Re: [Lldb-commits] [PATCH] D14937: swig_bot remote path preliminary implementation

2015-11-23 Thread Zachary Turner via lldb-commits
zturner updated this revision to Diff 40991. zturner added a comment. `socket.recv` doesn't guarantee a full read. The data can be truncated if it could not be read all at once. This patch fixes this by introducing a helper function in `lldbsuite.support` to read an exact number of bytes, retr

[Lldb-commits] Some buildbot statistics for week of 11/15/2015 - 11/21/2015

2015-11-23 Thread Galina Kistanova via lldb-commits
Hello everyone, Below are some buildbot numbers for the last week of 11/15/2015 - 11/21/2015. Thanks Galina Number of commits by project: project | commits ---+--- llvm | 265 cfe | 1

Re: [Lldb-commits] Some buildbot statistics for week of 11/15/2015 - 11/21/2015

2015-11-23 Thread Sean Silva via lldb-commits
In the future can you also attach the data in CSV format? On Mon, Nov 23, 2015 at 3:46 PM, Galina Kistanova via cfe-commits < cfe-comm...@lists.llvm.org> wrote: > Hello everyone, > > Below are some buildbot numbers for the last week of 11/15/2015 - > 11/21/2015. > > Thanks > > Galina > > > > Numb

Re: [Lldb-commits] Some buildbot statistics for the last week

2015-11-23 Thread Evgenii Stepanov via lldb-commits
Permanently failing bots are way better that 50/50 flaky bots, but still not good. As for the sanitizer-x86_64-linux-bootstrap, it shows some legitimate failures in different parts of llvm that no one bothered to fix. That's kinda understandable because MSan failures are hard to reproduce locally (

Re: [Lldb-commits] Some buildbot statistics for the last week

2015-11-23 Thread Aaron Ballman via lldb-commits
On Wed, Nov 18, 2015 at 1:40 PM, Galina Kistanova via cfe-commits wrote: > Hello everyone, > > Here are some buildbot statistics you may found interesting. I will be > adding more statistics. > My goal is to publish metrics to help with keeping the buildbot > infrastructure healthy and improving i