[Lldb-commits] [lldb] r247124 - Remove an invalid check in DW_OP_piece processing.

2015-09-09 Thread Hafiz Abid Qadeer via lldb-commits
Author: abidh Date: Wed Sep 9 04:06:05 2015 New Revision: 247124 URL: http://llvm.org/viewvc/llvm-project?rev=247124=rev Log: Remove an invalid check in DW_OP_piece processing. Summary: When lldb is processing a location containing DW_OP_piece, the result is being stored in the 'pieces'

Re: [Lldb-commits] [PATCH] D12674: Remove an invalid check in DW_OP_piece processing.

2015-09-09 Thread Hafiz Abid Qadeer via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL247124: Remove an invalid check in DW_OP_piece processing. (authored by abidh). Changed prior to commit: http://reviews.llvm.org/D12674?vs=34154=34301#toc Repository: rL LLVM

Re: [Lldb-commits] [PATCH] D12670: [LLDB][MIPS] MIPS load/store instruction emulation for hardware watchpoints

2015-09-09 Thread Mohit Bhakkad via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL247129: [LLDB][MIPS] MIPS load/store instruction emulation for hardware watchpoints (authored by mohit.bhakkad). Changed prior to commit: http://reviews.llvm.org/D12670?vs=34132=34308#toc Repository:

[Lldb-commits] [lldb] r247129 - [LLDB][MIPS] MIPS load/store instruction emulation for hardware watchpoints

2015-09-09 Thread Mohit K. Bhakkad via lldb-commits
Author: mohit.bhakkad Date: Wed Sep 9 05:17:58 2015 New Revision: 247129 URL: http://llvm.org/viewvc/llvm-project?rev=247129=rev Log: [LLDB][MIPS] MIPS load/store instruction emulation for hardware watchpoints Reviewers: clayborg. Subscribers: jaydeep, bhushan, sagar, nitesh.jain, lldb-commits.

[Lldb-commits] [lldb] r247135 - Add missing include after rL247131

2015-09-09 Thread Tamas Berghammer via lldb-commits
Author: tberghammer Date: Wed Sep 9 06:16:32 2015 New Revision: 247135 URL: http://llvm.org/viewvc/llvm-project?rev=247135=rev Log: Add missing include after rL247131 Modified: lldb/trunk/include/lldb/Core/MappedHash.h Modified: lldb/trunk/include/lldb/Core/MappedHash.h URL:

[Lldb-commits] TestFormattersBoolRefPtr.py on OSX

2015-09-09 Thread Tamas Berghammer via lldb-commits
Hi Enrico, TestFormattersBoolRefPtr.py started to fail on the OSX buildbot. I think the first build where it is failed is http://lab.llvm.org:8011/builders/lldb-x86_64-darwin-13.4/builds/5522, but I am not sure as the bot is a bit unreliable. Can you take a look for it, or if not, then are you

Re: [Lldb-commits] [PATCH] D12671: [LLDB][MIPS] Added support for the debugging of N32/O32 applications on MIPS64 target.

2015-09-09 Thread Mohit Bhakkad via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL247134: [LLDB][MIPS] Added support for the debugging of N32/O32 applications on… (authored by mohit.bhakkad). Changed prior to commit: http://reviews.llvm.org/D12671?vs=34146=34314#toc Repository:

[Lldb-commits] [lldb] r247131 - Code cleanup in preparation of adding split dwarf support

2015-09-09 Thread Tamas Berghammer via lldb-commits
Author: tberghammer Date: Wed Sep 9 05:20:36 2015 New Revision: 247131 URL: http://llvm.org/viewvc/llvm-project?rev=247131=rev Log: Code cleanup in preparation of adding split dwarf support * Remove some unused code * Remove usage of DWARFDebugInfoEntry::Attributes where usage isn't

[Lldb-commits] [lldb] r247130 - Remove the dwo files in the cleanup stage of the tests

2015-09-09 Thread Tamas Berghammer via lldb-commits
Author: tberghammer Date: Wed Sep 9 05:20:30 2015 New Revision: 247130 URL: http://llvm.org/viewvc/llvm-project?rev=247130=rev Log: Remove the dwo files in the cleanup stage of the tests The dwo files are generated when the tests run with split dwarf info. Modified:

Re: [Lldb-commits] [PATCH] D12677: Bug 24733: TestRegisters.py for Clang inferiors

2015-09-09 Thread Abhishek via lldb-commits
abhishek.aggarwal added a comment. In http://reviews.llvm.org/D12677#242006, @ovyalov wrote: > I reverted the CL because it was causing > TestRegisters.test_fp_special_purpose_register_read to fail on OSX: > > - stop reason = EXC_BREAKPOINT > - "register read ftag" yields 0x80 instead of

[Lldb-commits] [lldb] r247137 - Fix type in include header name

2015-09-09 Thread Tamas Berghammer via lldb-commits
Author: tberghammer Date: Wed Sep 9 06:28:52 2015 New Revision: 247137 URL: http://llvm.org/viewvc/llvm-project?rev=247137=rev Log: Fix type in include header name Modified: lldb/trunk/include/lldb/Core/MappedHash.h Modified: lldb/trunk/include/lldb/Core/MappedHash.h URL:

Re: [Lldb-commits] [PATCH] D12699: Change the looping stack detection code

2015-09-09 Thread Tamas Berghammer via lldb-commits
tberghammer added a comment. In ABI conforming code we will never have even 2 consecutive frames with the same CFA for all ABI I know about. In practice, 2 consecutive frame with the same CFA can happen (for hand written assembly and signal handling), but more then 2 is highly unlikely. In

[Lldb-commits] [lldb] r247133 - Change the looping stack detection code

2015-09-09 Thread Tamas Berghammer via lldb-commits
Author: tberghammer Date: Wed Sep 9 05:26:50 2015 New Revision: 247133 URL: http://llvm.org/viewvc/llvm-project?rev=247133=rev Log: Change the looping stack detection code In some special case (e.g. signal handlers, hand written assembly) it is valid to have 2 stack frame with the same CFA

[Lldb-commits] [lldb] r247141 - XFAIL TestFormatters on FreeBSD

2015-09-09 Thread Ed Maste via lldb-commits
Author: emaste Date: Wed Sep 9 08:10:33 2015 New Revision: 247141 URL: http://llvm.org/viewvc/llvm-project?rev=247141=rev Log: XFAIL TestFormatters on FreeBSD The test is hitting an assertion in Clang. This is an extension of r246766. llvm.org/pr24691 Modified:

[Lldb-commits] [lldb] r247202 - Fix the build-llvm.pl to not create one monster .a file from all of the llvm and clang .a files. We now just make a file list which we pass to the linker.

2015-09-09 Thread Greg Clayton via lldb-commits
Author: gclayton Date: Wed Sep 9 17:35:25 2015 New Revision: 247202 URL: http://llvm.org/viewvc/llvm-project?rev=247202=rev Log: Fix the build-llvm.pl to not create one monster .a file from all of the llvm and clang .a files. We now just make a file list which we pass to the linker. Modified:

[Lldb-commits] [lldb] r247195 - Remove Target::GetBasicType()

2015-09-09 Thread Enrico Granata via lldb-commits
Author: enrico Date: Wed Sep 9 17:00:18 2015 New Revision: 247195 URL: http://llvm.org/viewvc/llvm-project?rev=247195=rev Log: Remove Target::GetBasicType() I was experimenting with it briefly, and then settled on Target::GetTypeSystem + TypeSystem::GetBasicType, so this API is not necessary to

[Lldb-commits] [lldb] r247200 - Introduce the notion of an escape helper. Different languages have different notion of what to print in a string and how to escape non-printable things. The escape help

2015-09-09 Thread Enrico Granata via lldb-commits
Author: enrico Date: Wed Sep 9 17:30:24 2015 New Revision: 247200 URL: http://llvm.org/viewvc/llvm-project?rev=247200=rev Log: Introduce the notion of an escape helper. Different languages have different notion of what to print in a string and how to escape non-printable things. The escape

Re: [Lldb-commits] [lldb] r247082 - Implement a Target::GetTypeSystemForLanguage API, as well as provide helpers on the TypeSystem to get numeric types of specific sizes and signedness

2015-09-09 Thread Ryan Brown via lldb-commits
On Wed, Sep 9, 2015 at 2:57 PM Enrico Granata wrote: > On Sep 9, 2015, at 2:47 PM, Ryan Brown wrote: > > > On Tue, Sep 8, 2015 at 3:10 PM Enrico Granata via lldb-commits < > lldb-commits@lists.llvm.org> wrote: > >> Author: enrico >> Date: Tue Sep 8

[Lldb-commits] [lldb] r247206 - Fix a small comment typo in Windows Process code.

2015-09-09 Thread Stephane Sezer via lldb-commits
Author: sas Date: Wed Sep 9 17:58:23 2015 New Revision: 247206 URL: http://llvm.org/viewvc/llvm-project?rev=247206=rev Log: Fix a small comment typo in Windows Process code. Modified: lldb/trunk/source/Plugins/Process/Windows/Live/LocalDebugDelegate.h Modified:

[Lldb-commits] [PATCH] D12748: Include platform agnostic in the place of

2015-09-09 Thread Kamil Rytarowski via lldb-commits
krytarowski created this revision. krytarowski added a reviewer: joerg. krytarowski added a subscriber: lldb-commits. krytarowski set the repository for this revision to rL LLVM. Problem caught on NetBSD with missing getopt_long_only(3). Repository: rL LLVM http://reviews.llvm.org/D12748

[Lldb-commits] [PATCH] D12749: Link liblldb.so with LLVMObjCARCOpts.a

2015-09-09 Thread Kamil Rytarowski via lldb-commits
krytarowski created this revision. krytarowski added a reviewer: joerg. krytarowski added a subscriber: lldb-commits. krytarowski set the repository for this revision to rL LLVM. Problems reported on NetBSD: ``` llvm[4]: Linking Debug+Asserts executable lldb

[Lldb-commits] [lldb] r247224 - On MacOSX, revamp the way we link against the llvm/clang .a files by making a text file that contains all .a filenames and use that when linking in Xcode.

2015-09-09 Thread Greg Clayton via lldb-commits
Author: gclayton Date: Wed Sep 9 19:48:44 2015 New Revision: 247224 URL: http://llvm.org/viewvc/llvm-project?rev=247224=rev Log: On MacOSX, revamp the way we link against the llvm/clang .a files by making a text file that contains all .a filenames and use that when linking in Xcode. Modified:

[Lldb-commits] [PATCH] D12746: The pipe2(2) call is supported on NetBSD

2015-09-09 Thread Kamil Rytarowski via lldb-commits
krytarowski created this revision. krytarowski added a reviewer: joerg. krytarowski added a subscriber: lldb-commits. krytarowski set the repository for this revision to rL LLVM. Linux and NetBSD versions are compatible, modulo flag values: - O_CLOEXEC and O_NONBLOCK are the same, - O_NOSIGPIPE

Re: [Lldb-commits] [PATCH] D12659: Use SI_KERNEL on platforms defining it

2015-09-09 Thread Kamil Rytarowski via lldb-commits
krytarowski added a comment. Still valid. Repository: rL LLVM http://reviews.llvm.org/D12659 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D12079: [MIPS] microMIPS breakpoints, disassembly and compressed addresses

2015-09-09 Thread Jaydeep Patil via lldb-commits
jaydeep added a comment. In http://reviews.llvm.org/D12079#242751, @clayborg wrote: > Actually not a new format type, but an extra arg will need to be passed to > DumpAddress like "bool addr_is_callable". > > Can you explain something to me? In the following example: > > 0x8020067d <+0>:

Re: [Lldb-commits] [PATCH] D12115: [LLDB-MI] Fix -data-info-line and -symbol-list-lines when Windows filenames are used.

2015-09-09 Thread Zachary Turner via lldb-commits
I'm probably not the best person for this since I don't know anything about the MI stuff On Wed, Sep 9, 2015 at 8:33 PM Dawn Perchik wrote: > dawn added a comment. > > ping? > > > Repository: > rL LLVM > > http://reviews.llvm.org/D12115 > > > >

Re: [Lldb-commits] [PATCH] D12746: The pipe2(2) call is supported on NetBSD

2015-09-09 Thread Joerg Sonnenberger via lldb-commits
On Thu, Sep 10, 2015 at 01:15:43AM +, Kamil Rytarowski wrote: > Linux and NetBSD versions are compatible, modulo flag values: > - O_CLOEXEC and O_NONBLOCK are the same, > - O_NOSIGPIPE on NetBSD, > - O_DIRECT on Linux. I wonder why we don't just detect this via appropiate build test. There

[Lldb-commits] [lldb] r247189 - Preparatory work for letting language plugins help the StringPrinter with formatting special characters

2015-09-09 Thread Enrico Granata via lldb-commits
Author: enrico Date: Wed Sep 9 15:59:49 2015 New Revision: 247189 URL: http://llvm.org/viewvc/llvm-project?rev=247189=rev Log: Preparatory work for letting language plugins help the StringPrinter with formatting special characters Modified:

Re: [Lldb-commits] TestFormattersBoolRefPtr.py on OSX

2015-09-09 Thread Tamas Berghammer via lldb-commits
I attached the log for the failure (come from the build bot). About the double reporting, I seen it on one of the build bot, but we have some case when 2 different test have the same name what can explain the issue (e.g. TestFormattersBoolRefPtr) Tamas On Wed, Sep 9, 2015 at 6:54 PM Zachary

Re: [Lldb-commits] TestFormattersBoolRefPtr.py on OSX

2015-09-09 Thread Enrico Granata via lldb-commits
> On Sep 9, 2015, at 5:10 AM, Tamas Berghammer wrote: > > Hi Enrico, > > TestFormattersBoolRefPtr.py started to fail on the OSX buildbot. I think the > first build where it is failed is > http://lab.llvm.org:8011/builders/lldb-x86_64-darwin-13.4/builds/5522 >

Re: [Lldb-commits] [PATCH] D12728: New RenderScript command to break on all kernels

2015-09-09 Thread Jim Ingham via lldb-commits
jingham accepted this revision. jingham added a comment. This revision is now accepted and ready to land. This looks fine to me. You might want to add LIBLLDB_LOG_BREAKPOINT to your log creation, you can them together, and if I was tracking breakpoints with "log enable lldb breakpoint" I'd

[Lldb-commits] [lldb] r247193 - Remove a call to deleted function.

2015-09-09 Thread Greg Clayton via lldb-commits
Author: gclayton Date: Wed Sep 9 16:34:32 2015 New Revision: 247193 URL: http://llvm.org/viewvc/llvm-project?rev=247193=rev Log: Remove a call to deleted function. Modified: lldb/trunk/source/Plugins/Disassembler/llvm/DisassemblerLLVMC.cpp Modified:

Re: [Lldb-commits] [lldb] r247082 - Implement a Target::GetTypeSystemForLanguage API, as well as provide helpers on the TypeSystem to get numeric types of specific sizes and signedness

2015-09-09 Thread Ryan Brown via lldb-commits
On Tue, Sep 8, 2015 at 3:10 PM Enrico Granata via lldb-commits < lldb-commits@lists.llvm.org> wrote: > Author: enrico > Date: Tue Sep 8 17:09:19 2015 > New Revision: 247082 > > URL: http://llvm.org/viewvc/llvm-project?rev=247082=rev > Log: > Implement a Target::GetTypeSystemForLanguage API, as

Re: [Lldb-commits] [PATCH] D12079: [MIPS] microMIPS breakpoints, disassembly and compressed addresses

2015-09-09 Thread Greg Clayton via lldb-commits
clayborg requested changes to this revision. clayborg added a comment. This revision now requires changes to proceed. So DumpAddress() in FormatEntity.cpp is a generic "dump any address by describing it". You can't just change the code to suit your needs for MIPS. This address could be any

Re: [Lldb-commits] [PATCH] D12079: [MIPS] microMIPS breakpoints, disassembly and compressed addresses

2015-09-09 Thread Greg Clayton via lldb-commits
clayborg added a comment. Actually not a new format type, but an extra arg will need to be passed to DumpAddress like "bool addr_is_callable". Can you explain something to me? In the following example: 0x8020067d <+0>: addiusp -16 0x8020067f <+2>: sw $fp, 12($sp) 0x80200681 <+4>: