Re: [Lldb-commits] [PATCH] D14740: Define LLDB_DISABLE_CURSES as a fallback for missing curses(3) or panel(3)

2015-11-17 Thread Kamil Rytarowski via lldb-commits
krytarowski abandoned this revision. krytarowski added a comment. I resign from this change. I will just go for -DLLDB_DISABLE_CURSES from command line on NetBSD-7.0. Repository: rL LLVM http://reviews.llvm.org/D14740 ___ lldb-commits mailing

Re: [Lldb-commits] New llvm build server account request

2015-11-17 Thread Kamil Rytarowski via lldb-commits
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hello Ying, Regarding the NetBSD buildbot for LLDB one NetBSD we need a tweak. Please test for the existence (only on NetBSD) of a file /usr/include/panel.h. If it's missing please add '-DLLDB_DISABLE_CURSES' to CMake configure arguments. if [ !

Re: [Lldb-commits] [lldb] r253317 - Add Pythonic language binding wrapper generation script.

2015-11-17 Thread Zachary Turner via lldb-commits
Is there any way to have the step that runs the swig executable not be part of this script? Seems like running swig should be a build step, and running these scripts on the output of swig should be a step that follows On Mon, Nov 16, 2015 at 11:20 PM Todd Fiala via lldb-commits <

Re: [Lldb-commits] [lldb] r253317 - Add Pythonic language binding wrapper generation script.

2015-11-17 Thread Todd Fiala via lldb-commits
Yes, I am planning on separating out the usage of the swig output from the massaging of that into the build. Right now I'm just in phase 1, which is to clean up the existing process without modifying it, and adopting it on Xcode. Humorously, with our recent removal of the searching for swig in

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

2015-11-17 Thread Tamas Berghammer via lldb-commits
tberghammer added a comment. As far as I know the gdb remote protocol says that the registers in the 'p' packet should be displayed in target byte order, but the protocol isn't too well specified (and in my opinion target byte order is a silly decision). If we accept that the 'p' packet is in

[Lldb-commits] [lldb] r253346 - Added --find-swig option to allow searching for swig in path or other likely locations.

2015-11-17 Thread Todd Fiala via lldb-commits
Author: tfiala Date: Tue Nov 17 11:13:06 2015 New Revision: 253346 URL: http://llvm.org/viewvc/llvm-project?rev=253346=rev Log: Added --find-swig option to allow searching for swig in path or other likely locations. Modified: lldb/trunk/scripts/prepare_bindings.py Modified:

[Lldb-commits] [lldb] r253347 - Reinstate Xcode project usage of prepare_bindings.py, with new --find-swig option set.

2015-11-17 Thread Todd Fiala via lldb-commits
Author: tfiala Date: Tue Nov 17 11:13:09 2015 New Revision: 253347 URL: http://llvm.org/viewvc/llvm-project?rev=253347=rev Log: Reinstate Xcode project usage of prepare_bindings.py, with new --find-swig option set. Modified: lldb/trunk/lldb.xcodeproj/project.pbxproj Modified:

[Lldb-commits] [lldb] r253353 - Remove --swig-executable `which swig` from Xcode prepare_bindings.py call.

2015-11-17 Thread Todd Fiala via lldb-commits
Author: tfiala Date: Tue Nov 17 12:10:54 2015 New Revision: 253353 URL: http://llvm.org/viewvc/llvm-project?rev=253353=rev Log: Remove --swig-executable `which swig` from Xcode prepare_bindings.py call. This is no longer needed with --find-swig, and although innocuous on systems with swig in

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

2015-11-17 Thread Greg Clayton via lldb-commits
clayborg added a comment. GDB remote protocol specifies that register values are sent in target byte order. We shouldn't change this. A big endian system should not send things as little endian. That being said, the current register context assumes you have a buffer that can contain all

Re: [Lldb-commits] [lldb] r253317 - Add Pythonic language binding wrapper generation script.

2015-11-17 Thread Zachary Turner via lldb-commits
On Tue, Nov 17, 2015 at 8:18 AM Todd Fiala wrote: > Breaking out the binding generation into a separate step will also be > important for a couple reasons: > > * (from before) I want to eliminate the requirement for the vast majority > of the builds to have a swig on their

[Lldb-commits] [PATCH] D14765: Support unix-abstract-connect scheme as platform url in lldb testsuite

2015-11-17 Thread Ying Chen via lldb-commits
chying created this revision. chying added a reviewer: ovyalov. chying added a subscriber: lldb-commits. http://reviews.llvm.org/D14765 Files: packages/Python/lldbsuite/test/lldbtest.py packages/Python/lldbsuite/test/tools/lldb-server/gdbremote_testcase.py Index:

Re: [Lldb-commits] [lldb] r253317 - Add Pythonic language binding wrapper generation script.

2015-11-17 Thread Zachary Turner via lldb-commits
Moving this back over to the list since I'm sure others have some input here. Also +lldb-dev since it has more visibility than lldb-commits. On Tue, Nov 17, 2015 at 11:25 AM Zachary Turner wrote: > On Tue, Nov 17, 2015 at 8:18 AM Todd Fiala wrote: >

[Lldb-commits] [lldb] r253423 - Cleanup the type X list commands to use the new ForEach goodness

2015-11-17 Thread Enrico Granata via lldb-commits
Author: enrico Date: Tue Nov 17 19:37:49 2015 New Revision: 253423 URL: http://llvm.org/viewvc/llvm-project?rev=253423=rev Log: Cleanup the type X list commands to use the new ForEach goodness Modified: lldb/trunk/include/lldb/DataFormatters/DataVisualization.h

Re: [Lldb-commits] New llvm build server account request

2015-11-17 Thread Kamil Rytarowski via lldb-commits
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 It's fixed. It was my fault as I forgot to make updateScripts.sh available via PATH. Buildslave works properly. Thank you! On 18.11.2015 03:31, Kamil Rytarowski via lldb-commits wrote: > I've noted the first set of successfully built LLDB

Re: [Lldb-commits] [lldb] r253317 - Add Pythonic language binding wrapper generation script.

2015-11-17 Thread Todd Fiala via lldb-commits
Nothing concrete at the moment; however, it could be interesting to look at the clang community and see what could be done for llvm-based language implementations. The angle that I think would be interesting would be if we can generate bindings more effectively based on the in-depth understanding

Re: [Lldb-commits] [lldb] r253317 - Add Pythonic language binding wrapper generation script.

2015-11-17 Thread Bruce Mitchener via lldb-commits
Stepping one step back further in the thread ... On Wed, Nov 18, 2015 at 8:35 AM, Zachary Turner via lldb-commits < lldb-commits@lists.llvm.org> wrote: > Moving this back over to the list since I'm sure others have some input > here. Also +lldb-dev since it has more visibility than

Re: [Lldb-commits] [PATCH] D14765: Support unix-abstract-connect scheme as platform url in lldb testsuite

2015-11-17 Thread Ying Chen via lldb-commits
chying marked 2 inline comments as done. Comment at: packages/Python/lldbsuite/test/tools/lldb-server/gdbremote_testcase.py:69 @@ -66,1 +68,3 @@ +scheme, host = re.match(url_pattern, lldb.platform_url).groups() +if lldb.remote_platform_name ==

Re: [Lldb-commits] [PATCH] D14740: Define LLDB_DISABLE_CURSES as a fallback for missing curses(3) or panel(3)

2015-11-17 Thread Pavel Labath via lldb-commits
labath added a subscriber: labath. labath requested changes to this revision. labath added a reviewer: labath. labath added a comment. This revision now requires changes to proceed. I don't approve of automagic dependencies. With this patch it won't be possible to force curses usage. Even if you

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

2015-11-17 Thread Sagar Thakur via lldb-commits
sagar added a comment. Hi @tberghammer, I tried using RegisterValue::SetUInt() instead of RegisterValue::SetBytes(). When using RegisterValue::SetUInt() all register values we get are zero in case of mips32 big endian machine. The

[Lldb-commits] [lldb] r253364 - Move this back to a pid_t; this file is OSX only anyway, and the signedness of pid_t is actually a thing we want here

2015-11-17 Thread Enrico Granata via lldb-commits
Author: enrico Date: Tue Nov 17 13:35:26 2015 New Revision: 253364 URL: http://llvm.org/viewvc/llvm-project?rev=253364=rev Log: Move this back to a pid_t; this file is OSX only anyway, and the signedness of pid_t is actually a thing we want here Modified:

Re: [Lldb-commits] New llvm build server account request

2015-11-17 Thread Ying Chen via lldb-commits
Done. Please let me know if it works for you. Thanks, Ying On Tue, Nov 17, 2015 at 3:34 AM, Kamil Rytarowski wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA256 > > Hello Ying, > > Regarding the NetBSD buildbot for LLDB one NetBSD we need a tweak. > > Please test for the

[Lldb-commits] [lldb] r253383 - Cleanup indentation

2015-11-17 Thread Enrico Granata via lldb-commits
Author: enrico Date: Tue Nov 17 15:55:09 2015 New Revision: 253383 URL: http://llvm.org/viewvc/llvm-project?rev=253383=rev Log: Cleanup indentation Modified: lldb/trunk/source/Commands/CommandObjectType.cpp Modified: lldb/trunk/source/Commands/CommandObjectType.cpp URL:

[Lldb-commits] [lldb] r253381 - Cleanup the type X clear commands, so that one base class can implement all of them

2015-11-17 Thread Enrico Granata via lldb-commits
Author: enrico Date: Tue Nov 17 15:43:55 2015 New Revision: 253381 URL: http://llvm.org/viewvc/llvm-project?rev=253381=rev Log: Cleanup the type X clear commands, so that one base class can implement all of them Modified: lldb/trunk/source/Commands/CommandObjectType.cpp Modified: