Re: [Lldb-commits] [PATCH] D19273: Update Go OS Plugin for newer runtimes

2016-04-21 Thread Ryan Brown via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL267048: Update Go OS Plugin for newer runtimes. (authored by ribrdb). Changed prior to commit: http://reviews.llvm.org/D19273?vs=54427&id=54572#toc Repository: rL LLVM http://reviews.llvm.org/D19273

[Lldb-commits] [lldb] r267048 - Update Go OS Plugin for newer runtimes.

2016-04-21 Thread Ryan Brown via lldb-commits
Author: ribrdb Date: Thu Apr 21 15:57:28 2016 New Revision: 267048 URL: http://llvm.org/viewvc/llvm-project?rev=267048&view=rev Log: Update Go OS Plugin for newer runtimes. Differential Revision: http://reviews.llvm.org/D19273 Modified: lldb/trunk/source/Plugins/OperatingSystem/Go/OperatingS

Re: [Lldb-commits] [PATCH] D19273: Update Go OS Plugin for newer runtimes

2016-04-20 Thread Ryan Brown via lldb-commits
ribrdb removed rL LLVM as the repository for this revision. ribrdb updated this revision to Diff 54427. http://reviews.llvm.org/D19273 Files: source/Plugins/OperatingSystem/Go/OperatingSystemGo.cpp Index: source/Plugins/OperatingSystem/Go/OperatingSystemGo.cpp =

Re: [Lldb-commits] [PATCH] D19273: Update Go OS Plugin for newer runtimes

2016-04-20 Thread Ryan Brown via lldb-commits
ribrdb marked an inline comment as done. ribrdb added a comment. I'm not sure how else we could identify the runtime version. I'm hoping to eventually add a producer string that has the version, but this would just be a git commit for development compilers. Repository: rL LLVM http://review

[Lldb-commits] [PATCH] D19273: Update Go OS Plugin for newer runtimes

2016-04-19 Thread Ryan Brown via lldb-commits
ribrdb created this revision. ribrdb added a subscriber: lldb-commits. ribrdb set the repository for this revision to rL LLVM. Repository: rL LLVM http://reviews.llvm.org/D19273 Files: source/Plugins/OperatingSystem/Go/OperatingSystemGo.cpp Index: source/Plugins/OperatingSystem/Go/Operating

Re: [Lldb-commits] [lldb] r257926 - Implement missing GoASTContext methods

2016-01-15 Thread Ryan Brown via lldb-commits
It would be nice if we could get this into 3.8 to get rid of these asserts. -- Ryan Brown On Fri, Jan 15, 2016 at 11:35 AM, Ryan Brown via lldb-commits < lldb-commits@lists.llvm.org> wrote: > Author: ribrdb > Date: Fri Jan 15 13:35:48 2016 > New Revision: 257926 > > URL: h

[Lldb-commits] [lldb] r257926 - Implement missing GoASTContext methods

2016-01-15 Thread Ryan Brown via lldb-commits
Author: ribrdb Date: Fri Jan 15 13:35:48 2016 New Revision: 257926 URL: http://llvm.org/viewvc/llvm-project?rev=257926&view=rev Log: Implement missing GoASTContext methods Modified: lldb/trunk/source/Symbol/GoASTContext.cpp Modified: lldb/trunk/source/Symbol/GoASTContext.cpp URL: http://llv

[Lldb-commits] [PATCH] D16234: Implement missing GoASTContext methods

2016-01-15 Thread Ryan Brown via lldb-commits
ribrdb created this revision. ribrdb added a reviewer: clayborg. ribrdb added a subscriber: lldb-commits. ribrdb set the repository for this revision to rL LLVM. Repository: rL LLVM http://reviews.llvm.org/D16234 Files: source/Symbol/GoASTContext.cpp Index: source/Symbol/GoASTContext.cpp ==

Re: [Lldb-commits] [PATCH] D13878: Add data formatters for go strings and slices.

2015-11-04 Thread Ryan Brown via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL252109: Add go data formatters. (authored by ribrdb). Changed prior to commit: http://reviews.llvm.org/D13878?vs=37900&id=39288#toc Repository: rL LLVM http://reviews.llvm.org/D13878 Files: lldb/t

[Lldb-commits] [lldb] r252109 - Add go data formatters.

2015-11-04 Thread Ryan Brown via lldb-commits
Author: ribrdb Date: Wed Nov 4 18:24:36 2015 New Revision: 252109 URL: http://llvm.org/viewvc/llvm-project?rev=252109&view=rev Log: Add go data formatters. Differential Revision: http://reviews.llvm.org/D13878 Added: lldb/trunk/packages/Python/lldbsuite/test/lang/go/formatters/ lldb/tr

[Lldb-commits] [lldb] r252007 - Fix build for go parser unittest.

2015-11-03 Thread Ryan Brown via lldb-commits
Author: ribrdb Date: Tue Nov 3 16:46:37 2015 New Revision: 252007 URL: http://llvm.org/viewvc/llvm-project?rev=252007&view=rev Log: Fix build for go parser unittest. Modified: lldb/trunk/lldb.xcodeproj/project.pbxproj lldb/trunk/unittests/Expression/GoParserTest.cpp Modified: lldb/trunk

[Lldb-commits] [lldb] r251835 - Move go expression tests to the new location.

2015-11-02 Thread Ryan Brown via lldb-commits
Author: ribrdb Date: Mon Nov 2 15:28:18 2015 New Revision: 251835 URL: http://llvm.org/viewvc/llvm-project?rev=251835&view=rev Log: Move go expression tests to the new location. Added: lldb/trunk/packages/Python/lldbsuite/test/lang/go/expressions/ lldb/trunk/packages/Python/lldbsuite/te

Re: [Lldb-commits] [PATCH] D13878: Add data formatters for go strings and slices.

2015-10-29 Thread Ryan Brown via lldb-commits
ribrdb added a comment. Is this ready to submit? http://reviews.llvm.org/D13878 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D13073: Add an expression parser for Go

2015-10-29 Thread Ryan Brown via lldb-commits
ribrdb added a comment. Is this ready to submit? Comment at: source/Expression/LLVMUserExpression.cpp:43-60 @@ +42,20 @@ + +LLVMUserExpression::LLVMUserExpression(ExecutionContextScope &exe_scope, const char *expr, const char *expr_prefix, +

Re: [Lldb-commits] [PATCH] D13878: Add data formatters for go strings and slices.

2015-10-20 Thread Ryan Brown via lldb-commits
ribrdb removed rL LLVM as the repository for this revision. ribrdb updated this revision to Diff 37900. ribrdb added a comment. Oops, I ran clang-format with the wrong arguments. Ran it again. http://reviews.llvm.org/D13878 Files: cmake/LLDBDependencies.cmake lldb.xcodeproj/project.pbxproj

Re: [Lldb-commits] [PATCH] D13878: Add data formatters for go strings and slices.

2015-10-20 Thread Ryan Brown via lldb-commits
ribrdb updated this revision to Diff 37898. Repository: rL LLVM http://reviews.llvm.org/D13878 Files: cmake/LLDBDependencies.cmake lldb.xcodeproj/project.pbxproj source/API/SystemInitializerFull.cpp source/Plugins/Language/CMakeLists.txt source/Plugins/Language/Go/CMakeLists.txt so

Re: [Lldb-commits] [PATCH] D13878: Add data formatters for go strings and slices.

2015-10-20 Thread Ryan Brown via lldb-commits
ribrdb added inline comments. Comment at: source/Plugins/Language/Go/GoFormatterFunctions.cpp:10 @@ +9,3 @@ + +#include + Eugene.Zelenko wrote: > Please add include section comments. Same for other sections. Which sections? I've seen a couple types of section mar

Re: [Lldb-commits] [PATCH] D13878: Add data formatters for go strings and slices.

2015-10-19 Thread Ryan Brown via lldb-commits
ribrdb updated this revision to Diff 37800. ribrdb added a comment. Add tests. Repository: rL LLVM http://reviews.llvm.org/D13878 Files: cmake/LLDBDependencies.cmake lldb.xcodeproj/project.pbxproj source/API/SystemInitializerFull.cpp source/Plugins/Language/CMakeLists.txt source/Pl

[Lldb-commits] [PATCH] D13878: Add data formatters for go strings and slices.

2015-10-19 Thread Ryan Brown via lldb-commits
ribrdb created this revision. ribrdb added a reviewer: clayborg. ribrdb added a subscriber: lldb-commits. ribrdb set the repository for this revision to rL LLVM. Repository: rL LLVM http://reviews.llvm.org/D13878 Files: cmake/LLDBDependencies.cmake lldb.xcodeproj/project.pbxproj source/A

Re: [Lldb-commits] [PATCH] D13149: Create GoLanguageRuntime

2015-10-06 Thread Ryan Brown via lldb-commits
ribrdb closed this revision. ribrdb added a comment. Submitted in revisions 249456 and 249459. Repository: rL LLVM http://reviews.llvm.org/D13149 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/list

[Lldb-commits] [lldb] r249478 - Simple readline functionality for interactive python on linux.

2015-10-06 Thread Ryan Brown via lldb-commits
Author: ribrdb Date: Tue Oct 6 17:21:08 2015 New Revision: 249478 URL: http://llvm.org/viewvc/llvm-project?rev=249478&view=rev Log: Simple readline functionality for interactive python on linux. Differential Revision: http://reviews.llvm.org/D13268 Modified: lldb/trunk/CMakeLists.txt ll

[Lldb-commits] [lldb] r249459 - Add missing GoLanguageRuntime files.

2015-10-06 Thread Ryan Brown via lldb-commits
Author: ribrdb Date: Tue Oct 6 15:31:08 2015 New Revision: 249459 URL: http://llvm.org/viewvc/llvm-project?rev=249459&view=rev Log: Add missing GoLanguageRuntime files. Added: lldb/trunk/source/Plugins/LanguageRuntime/Go/ lldb/trunk/source/Plugins/LanguageRuntime/Go/CMakeLists.txt ll

[Lldb-commits] [lldb] r249456 - Create GoLanguageRuntime.

2015-10-06 Thread Ryan Brown via lldb-commits
Author: ribrdb Date: Tue Oct 6 15:29:31 2015 New Revision: 249456 URL: http://llvm.org/viewvc/llvm-project?rev=249456&view=rev Log: Create GoLanguageRuntime. GoLanguageRuntime supports finding the runtime type for Go interfaces. Modified: lldb/trunk/cmake/LLDBDependencies.cmake lldb/tru

Re: [Lldb-commits] [lldb] r249446 - Address failing Go tests on go version from Ubuntu 14.04

2015-10-06 Thread Ryan Brown via lldb-commits
I was wondering if we'd need this. Go 1.4 is the minimum supported version. On Tue, Oct 6, 2015 at 12:17 PM Todd Fiala via lldb-commits < lldb-commits@lists.llvm.org> wrote: > Author: tfiala > Date: Tue Oct 6 14:15:56 2015 > New Revision: 249446 > > URL: http://llvm.org/viewvc/llvm-project?rev=2

Re: [Lldb-commits] [PATCH] D13268: Simple readline functionality for interactive python on linux.

2015-10-06 Thread Ryan Brown via lldb-commits
ribrdb updated this revision to Diff 36637. Repository: rL LLVM http://reviews.llvm.org/D13268 Files: CMakeLists.txt scripts/Python/modules/readline/CMakeLists.txt scripts/Python/modules/readline/readline.cpp source/CMakeLists.txt Index: source/CMakeLists.txt =

Re: [Lldb-commits] [PATCH] D13268: Simple readline functionality for interactive python on linux.

2015-10-01 Thread Ryan Brown via lldb-commits
ribrdb updated this revision to Diff 36295. ribrdb marked an inline comment as done. ribrdb added a comment. Updated comments. Repository: rL LLVM http://reviews.llvm.org/D13268 Files: scripts/Python/modules/readline/CMakeLists.txt scripts/Python/modules/readline/readline.cpp Index: scr

Re: [Lldb-commits] [lldb] r248934 - Now persistent expression data no longer lives with the Target, but rather with

2015-10-01 Thread Ryan Brown via lldb-commits
Oh, now I see the search in 249027. But how does this work with multiple type systems? Target no longer manages persistent variable names so won't each type system generate conflicting names? e.g. (lldb) expr -l c -- 1 $1 = 1 (lldb) expr -l go -- 2 $1 = 2 (lldb) expr -l go -- $1 Which $1 do you ge

Re: [Lldb-commits] [lldb] r248934 - Now persistent expression data no longer lives with the Target, but rather with

2015-10-01 Thread Ryan Brown via lldb-commits
Shouldn't Target::GetPersistentVariable take a language parameter instead of assuming clang? On Wed, Sep 30, 2015 at 12:59 PM Sean Callanan via lldb-commits < lldb-commits@lists.llvm.org> wrote: > Author: spyffe > Date: Wed Sep 30 14:57:57 2015 > New Revision: 248934 > > URL: http://llvm.org/view

Re: [Lldb-commits] [PATCH] D13149: Create GoLanguageRuntime

2015-09-29 Thread Ryan Brown via lldb-commits
ribrdb updated this revision to Diff 36031. ribrdb added a comment. Fix cmake build. Repository: rL LLVM http://reviews.llvm.org/D13149 Files: cmake/LLDBDependencies.cmake lldb.xcodeproj/project.pbxproj source/API/SystemInitializerFull.cpp source/Plugins/LanguageRuntime/CMakeLists.tx

[Lldb-commits] [PATCH] D13268: Simple readline functionality for interactive python on linux.

2015-09-29 Thread Ryan Brown via lldb-commits
ribrdb created this revision. ribrdb added a reviewer: tfiala. ribrdb added a subscriber: lldb-commits. ribrdb set the repository for this revision to rL LLVM. This implements basic line editing for the interactive python interpreter on linux. No tab completion, but at least backspace works. Rep

[Lldb-commits] [PATCH] D13149: Create GoLanguageRuntime

2015-09-24 Thread Ryan Brown via lldb-commits
ribrdb created this revision. ribrdb added a reviewer: clayborg. ribrdb added a subscriber: lldb-commits. ribrdb set the repository for this revision to rL LLVM. GoLanguageRuntime supports finding the runtime type for Go interfaces. Repository: rL LLVM http://reviews.llvm.org/D13149 Files:

Re: [Lldb-commits] [PATCH] D13073: Add an expression parser for Go

2015-09-23 Thread Ryan Brown via lldb-commits
ribrdb added a comment. I suppose JIT might make more sense if we want to implement more complicated expressions like looping and calling in to go code. There's lots of issues to solve before we can call into go code however. In addition to the stack I'm not sure what to do about the GC. Also I

Re: [Lldb-commits] [PATCH] D13073: Add an expression parser for Go

2015-09-23 Thread Ryan Brown via lldb-commits
ribrdb added a comment. Hmm. I assumed you're using clang to generate llvm IR, is that not the case? I don't really want to write a whole go compiler. Right now I'm interpreting the AST using the ValueObject API. This also lets me reuse the error checking and things implemented in the type system

Re: [Lldb-commits] [lldb] r248366 - Testcase and fix for bug 24074

2015-09-23 Thread Ryan Brown via lldb-commits
The doc comment for SortTypeList() is wrong. On Sep 23, 2015 12:20 AM, "Ravitheja Addepally via lldb-commits" < lldb-commits@lists.llvm.org> wrote: > Author: ravitheja > Date: Wed Sep 23 02:19:02 2015 > New Revision: 248366 > > URL: http://llvm.org/viewvc/llvm-project?rev=248366&view=rev > Log: >

[Lldb-commits] [lldb] r247852 - Add an OperatingSystem plugin to support goroutines

2015-09-16 Thread Ryan Brown via lldb-commits
Author: ribrdb Date: Wed Sep 16 16:20:44 2015 New Revision: 247852 URL: http://llvm.org/viewvc/llvm-project?rev=247852&view=rev Log: Add an OperatingSystem plugin to support goroutines The Go runtime schedules user level threads (goroutines) across real threads. This adds an OS plugin to create m

Re: [Lldb-commits] [PATCH] D5871: Add an OperatingSystem plugin to support goroutines

2015-09-16 Thread Ryan Brown via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL247852: Add an OperatingSystem plugin to support goroutines (authored by ribrdb). Changed prior to commit: http://reviews.llvm.org/D5871?vs=34914&id=34925#toc Repository: rL LLVM http://reviews.llvm

Re: [Lldb-commits] [PATCH] D5871: Add an OperatingSystem plugin to support goroutines

2015-09-16 Thread Ryan Brown via lldb-commits
ribrdb marked 2 inline comments as done. ribrdb added a comment. Repository: rL LLVM http://reviews.llvm.org/D5871 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D5871: Add an OperatingSystem plugin to support goroutines

2015-09-16 Thread Ryan Brown via lldb-commits
ribrdb updated this revision to Diff 34914. ribrdb added a comment. > I am confused as to why we need to do this. If you have a memory thread, it > might be backed by a real thread, or it might not. If it is backed by a > real thread, we should be asking the real thread why it really stopp

Re: [Lldb-commits] [PATCH] D5871: Add an OperatingSystem plugin to support goroutines

2015-09-15 Thread Ryan Brown via lldb-commits
ribrdb updated this revision to Diff 34839. ribrdb added a comment. Fix cmake build. Repository: rL LLVM http://reviews.llvm.org/D5871 Files: cmake/LLDBDependencies.cmake include/lldb/Core/PluginManager.h lib/Makefile lldb.xcodeproj/project.pbxproj source/Core/PluginManager.cpp s

Re: [Lldb-commits] [PATCH] D5871: Add an OperatingSystem plugin to support goroutines

2015-09-15 Thread Ryan Brown via lldb-commits
ribrdb updated this revision to Diff 34813. ribrdb added a comment. Ok, I've changed it to use the symbol table like the ASAN runtime does. I'm not seeing any language info in the elf or macho file headers. It looks like there's also a gosymtab section I could look for, but the name is different

Re: [Lldb-commits] [PATCH] D5871: Add an OperatingSystem plugin to support goroutines

2015-09-15 Thread Ryan Brown via lldb-commits
ribrdb updated the summary for this revision. ribrdb set the repository for this revision to rL LLVM. ribrdb updated this revision to Diff 34806. ribrdb added a comment. I've updated this to load the plugin when modules are loaded, added a setting to enable/disable the goroutine plugin, and added

Re: [Lldb-commits] [PATCH] D12872: Use uint64_t for GoArray size.

2015-09-14 Thread Ryan Brown via lldb-commits
ribrdb accepted this revision. ribrdb added a comment. This revision is now accepted and ready to land. Looks good http://reviews.llvm.org/D12872 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinf

[Lldb-commits] [lldb] r247647 - Fix error on windows:

2015-09-14 Thread Ryan Brown via lldb-commits
Author: ribrdb Date: Mon Sep 14 19:50:43 2015 New Revision: 247647 URL: http://llvm.org/viewvc/llvm-project?rev=247647&view=rev Log: Fix error on windows: c:\buildbot\lldb-windows-x86\lldb-windows-x86\llvm\tools\lldb\source\symbol\goastcontext.cpp(906) : warning C4715: 'lldb_private::GoASTContex

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 17:09:19 2015 >> New Revision: 247082 >> >> UR

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&view=rev > Log: > Implement a Target::GetTypeSystemForLanguage API,