Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-27 Thread Zachary Turner via lldb-commits
Any reason dotest / dosep cannot just print the stats? On Thu, Aug 27, 2015 at 4:17 PM Todd Fiala via lldb-commits < lldb-commits@lists.llvm.org> wrote: > tfiala updated this revision to Diff 33375. > > http://reviews.llvm.org/D12416 > > Files: > test/reports/dotest_stats.py > > ___

Re: [Lldb-commits] [PATCH] D12415: Use $ to get the path lf lldb in test runs.

2015-08-27 Thread Zachary Turner via lldb-commits
I think it's still worth printing something, even if it's just "Testing LLDB with N processes", where you fill in the value for N On Thu, Aug 27, 2015 at 4:07 PM Stephane Sezer wrote: > sas added a comment. > > I removed the "Testing..." messages because $ won't be > expanded in the context of a

Re: [Lldb-commits] [PATCH] D12363: Read module list from mini dump

2015-08-27 Thread Zachary Turner via lldb-commits
looks good On Thu, Aug 27, 2015 at 4:43 PM Adrian McCarthy wrote: > amccarth updated this revision to Diff 33379. > amccarth added a comment. > > Now uses llvm/Support/ConvertUTF.h instead of WinAPI WideCharToMultiByte. > > > http://reviews.llvm.org/D12363 > > Files: > source/Plugins/Process/W

[Lldb-commits] [PATCH] D12420: Make ProcessGDBRemote get a //copy// of platform Unix signals.

2015-08-27 Thread Chaoren Lin via lldb-commits
chaoren created this revision. chaoren added reviewers: clayborg, jaydeep. chaoren added a subscriber: lldb-commits. Update to http://reviews.llvm.org/rL243618. http://reviews.llvm.org/D12420 Files: source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp Index: source/Plugins/Process/gdb-remot

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread Greg Clayton via lldb-commits
Try this out: % svn commit Sendinginclude/lldb/Symbol/ClangASTContext.h Sendinginclude/lldb/Symbol/TypeSystem.h Sendinglldb.xcodeproj/project.pbxproj Sendingsource/Plugins/SymbolFile/DWARF/CMakeLists.txt Adding source/Plugins/SymbolFile/DWARF/DWARFASTParser.

Re: [Lldb-commits] [PATCH] D12363: Read module list from mini dump

2015-08-27 Thread Adrian McCarthy via lldb-commits
amccarth updated this revision to Diff 33379. amccarth added a comment. Now uses llvm/Support/ConvertUTF.h instead of WinAPI WideCharToMultiByte. http://reviews.llvm.org/D12363 Files: source/Plugins/Process/Windows/MiniDump/ProcessWinMiniDump.cpp source/Plugins/Process/Windows/MiniDump/Proc

Re: [Lldb-commits] [PATCH] D12363: Read module list from mini dump

2015-08-27 Thread Adrian McCarthy via lldb-commits
amccarth marked an inline comment as done. amccarth added a comment. New patch coming up. http://reviews.llvm.org/D12363 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D11102: Set the default language to use when evaluating to that of the frame's CU.

2015-08-27 Thread Sean Callanan via lldb-commits
spyffe added a comment. I looked at this with Jim, and once his suggestions are applied this patch is fine. Thanks for working with us on it. I want to follow up and make non-Apple platforms not force ObjC (it doesn't matter for making the expression parser work). http://reviews.llvm.org/D111

Re: [Lldb-commits] [PATCH] D11102: Set the default language to use when evaluating to that of the frame's CU.

2015-08-27 Thread Jim Ingham via lldb-commits
jingham requested changes to this revision. jingham added a comment. This revision now requires changes to proceed. This seems fine. The only things you should fix (see inline comments) are the name of GetDefaultExpressionLanguage and the comment on why the upgrading is necessary. If you do th

Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-27 Thread Todd Fiala via lldb-commits
tfiala updated this revision to Diff 33375. http://reviews.llvm.org/D12416 Files: test/reports/dotest_stats.py Index: test/reports/dotest_stats.py === --- test/reports/dotest_stats.py +++ test/reports/dotest_stats.py @@ -0,0 +1,14

Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-27 Thread Todd Fiala via lldb-commits
tfiala added a comment. And, the file is actually in test/reports/, not reports/... http://reviews.llvm.org/D12416 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-27 Thread Todd Fiala via lldb-commits
tfiala added a comment. Note the output is formatted for whitespace and left justified, with the numeric columns all lined up. My verbatim mode above doesn't look like it did what I expected... http://reviews.llvm.org/D12416 ___ lldb-commits mail

[Lldb-commits] [PATCH] D12416: Parse dotest.py/dosep.py output, providing general stats and skip reason breakdown by counts.

2015-08-27 Thread Todd Fiala via lldb-commits
tfiala created this revision. tfiala added a reviewer: clayborg. tfiala added a subscriber: lldb-commits. The new script can be run against output of dotest.py/dosep.py, with the primary benefit of providing a detailed breakdown of the number of test methods that skip for a given reason. Output

Re: [Lldb-commits] [PATCH] D12415: Use $ to get the path lf lldb in test runs.

2015-08-27 Thread Stephane Sezer via lldb-commits
sas added a comment. I removed the "Testing..." messages because $ won't be expanded in the context of a log message. Basically, you'll end up with ninja/msvc/make printing "Testing LLDB with args: --arch=x86_64;--executable;$;-s;/..." (even though it will be running the right command with the

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread via lldb-commits
On Thu, Aug 27, 2015 at 03:57:30PM -0700, Greg Clayton wrote: > > I can probably clean this up by making a DWARFASTParser a plug-in type and > move all DWARF parsing code over into it and place that inside the DWARF > plug-in folder. I see if I can do that quick to get the code organization > b

Re: [Lldb-commits] [PATCH] D12415: Use $ to get the path lf lldb in test runs.

2015-08-27 Thread Zachary Turner via lldb-commits
I'm not familiar with that $ expression, but if you're certain that it's equivalent to what was there previously, this looks fine. But why were the two "Testing LLDB..." messages removed? On Thu, Aug 27, 2015 at 3:55 PM Stephane Sezer wrote: > sas created this revision. > sas added reviewers: z

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread Greg Clayton via lldb-commits
> On Aug 27, 2015, at 2:12 PM, Zachary Turner wrote: > > If I'm following this discussion correctly, Greg are you saying that > ClangASTContext could be renamed to DwarfASTContext? And that for other > types of Debug info we could have PascalASTContext and PdbASTContext? No. ClangASTContext

[Lldb-commits] [PATCH] D12415: Use $ to get the path lf lldb in test runs.

2015-08-27 Thread Stephane Sezer via lldb-commits
sas created this revision. sas added reviewers: zturner, clayborg. sas added a subscriber: lldb-commits. Instead of hardcoding the path, we should use $ which works better when the output path differs, e.g. when building with MSVC. $ is expanded during the generation of the final build files, so

Re: [Lldb-commits] [PATCH] D11102: Set the default language to use when evaluating to that of the frame's CU.

2015-08-27 Thread Dawn Perchik via lldb-commits
dawn added a comment. In http://reviews.llvm.org/D11102#234450, @clayborg wrote: > Please wait for the OK from Jim and Sean. Jim/Sean, you guys OK with this? All tests pass - we've been running with this patch locally since it was proposed and have had no issues with it. http://reviews.llvm

[Lldb-commits] [lldb] r246222 - Include

2015-08-27 Thread Enrico Granata via lldb-commits
Author: enrico Date: Thu Aug 27 17:14:06 2015 New Revision: 246222 URL: http://llvm.org/viewvc/llvm-project?rev=246222&view=rev Log: Include Modified: lldb/trunk/source/Target/Language.cpp Modified: lldb/trunk/source/Target/Language.cpp URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/s

[Lldb-commits] [lldb] r246220 - Fixup one of the CMakeLists

2015-08-27 Thread Enrico Granata via lldb-commits
Author: enrico Date: Thu Aug 27 16:55:55 2015 New Revision: 246220 URL: http://llvm.org/viewvc/llvm-project?rev=246220&view=rev Log: Fixup one of the CMakeLists Modified: lldb/trunk/source/Plugins/Language/ObjC/CMakeLists.txt Modified: lldb/trunk/source/Plugins/Language/ObjC/CMakeLists.txt U

Re: [Lldb-commits] [PATCH] D12363: Read module list from mini dump

2015-08-27 Thread Zachary Turner via lldb-commits
zturner added inline comments. Comment at: source/Plugins/Process/Windows/MiniDump/ProcessWinMiniDump.cpp:57-63 @@ +56,9 @@ +const auto length_required = +::WideCharToMultiByte(CP_UTF8, 0, md_string->Buffer, md_string->Length, + nullptr, 0,

[Lldb-commits] [lldb] r246216 - Added a missing file to the CMakeLists

2015-08-27 Thread Enrico Granata via lldb-commits
Author: enrico Date: Thu Aug 27 16:45:59 2015 New Revision: 246216 URL: http://llvm.org/viewvc/llvm-project?rev=246216&view=rev Log: Added a missing file to the CMakeLists Modified: lldb/trunk/source/Target/CMakeLists.txt Modified: lldb/trunk/source/Target/CMakeLists.txt URL: http://llvm.or

[Lldb-commits] [lldb] r246212 - Add a new type of plugin: Language plugin

2015-08-27 Thread Enrico Granata via lldb-commits
Author: enrico Date: Thu Aug 27 16:33:50 2015 New Revision: 246212 URL: http://llvm.org/viewvc/llvm-project?rev=246212&view=rev Log: Add a new type of plugin: Language plugin The Language plugin is menat to answer language-specific questions that are not bound to the existence of a process. Thos

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread Zachary Turner via lldb-commits
What I'm getting at is that, assuming my understanding is correct, something like an adapter could be created that each plugin could implement. Say you've got a DebugInfoAdapter interface with DebugInfoDwarf, etc implementations. Then ClangASTContext could just use this. On Thu, Aug 27, 2015 at

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread via lldb-commits
On Thu, Aug 27, 2015 at 09:12:06PM +, Zachary Turner wrote: > If I'm following this discussion correctly, Greg are you saying that > ClangASTContext could be renamed to DwarfASTContext? And that for other > types of Debug info we could have PascalASTContext and PdbASTContext? > > If that's th

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread Zachary Turner via lldb-commits
If I'm following this discussion correctly, Greg are you saying that ClangASTContext could be renamed to DwarfASTContext? And that for other types of Debug info we could have PascalASTContext and PdbASTContext? If that's the case, then it seems there is a lot of code inside of ClangASTContext (ak

Re: [Lldb-commits] [PATCH] D11102: Set the default language to use when evaluating to that of the frame's CU.

2015-08-27 Thread via lldb-commits
On Thu, Aug 27, 2015 at 07:58:50PM +, Greg Clayton wrote: > clayborg added a comment. > > Please wait for the OK from Jim and Sean. I've been waiting for 1.5 months already. ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread via lldb-commits
On Thu, Aug 27, 2015 at 12:56:57PM -0700, Greg Clayton via lldb-commits wrote: > > > > Our Delphi compiler generates an alternate debug format as well - it took > > me all of yesterday to fix merge conflicts and I've still not got things > > working properly. Greg, can you please move the DWARF

Re: [Lldb-commits] [PATCH] D12304: Add scope tree for variable searching

2015-08-27 Thread Greg Clayton via lldb-commits
clayborg requested changes to this revision. clayborg added a comment. This revision now requires changes to proceed. This seems to be very variable specific. I would like to see if be able to ask a lldb_private::Block or a lldb_private::Compile and to provide a list of CompilerDeclContext objec

Re: [Lldb-commits] [PATCH] D11102: Set the default language to use when evaluating to that of the frame's CU.

2015-08-27 Thread Greg Clayton via lldb-commits
clayborg added a subscriber: clayborg. clayborg accepted this revision. clayborg added a reviewer: clayborg. clayborg added a comment. This revision is now accepted and ready to land. Looks fine as long as the test suite tests are passing. http://reviews.llvm.org/D11102 __

Re: [Lldb-commits] [PATCH] D11102: Set the default language to use when evaluating to that of the frame's CU.

2015-08-27 Thread Greg Clayton via lldb-commits
clayborg added a comment. Please wait for the OK from Jim and Sean. http://reviews.llvm.org/D11102 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread Greg Clayton via lldb-commits
> > Our Delphi compiler generates an alternate debug format as well - it took me > all of yesterday to fix merge conflicts and I've still not got things working > properly. Greg, can you please move the DWARF dependencies back into the > SymbolFileDWARF plugin? Thanks! Sorry to not be clear

Re: [Lldb-commits] [PATCH] D11102: Set the default language to use when evaluating to that of the frame's CU.

2015-08-27 Thread Dawn Perchik via lldb-commits
dawn added a comment. Hi guys, I'd still really like to get this into tot. May I commit it please? http://reviews.llvm.org/D11102 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread Dawn Perchik via lldb-commits
dawn raised a concern with this commit. dawn added a comment. Our Delphi compiler generates an alternate debug format as well - it took me all of yesterday to fix merge conflicts and I've still not got things working properly. Greg, can you please move the DWARF dependencies back into the Symb

Re: [Lldb-commits] [PATCH] D12304: Add scope tree for variable searching

2015-08-27 Thread Paul Herman via lldb-commits
paulherman updated this revision to Diff 33345. paulherman added a comment. Add scope tree for variable searching http://reviews.llvm.org/D12304 Files: include/lldb/Symbol/Block.h include/lldb/Symbol/CompileUnit.h include/lldb/Symbol/Function.h include/lldb/Symbol/Symbol.h include/lld

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread Greg Clayton via lldb-commits
> On Aug 26, 2015, at 9:08 PM, Dawn Perchik via lldb-commits > wrote: > > dawn added a subscriber: dawn. > dawn added a comment. > > Hi Greg, > > While I really appreciate that you're making lldb less Clang specific (e.g. > renaming ClangASTType to CompilerType), I'm concerned about this com

[Lldb-commits] [lldb] r246177 - Remove class Language - the only thing it was actually being used for is provided by LanguageRuntime already

2015-08-27 Thread Enrico Granata via lldb-commits
Author: enrico Date: Thu Aug 27 13:18:49 2015 New Revision: 246177 URL: http://llvm.org/viewvc/llvm-project?rev=246177&view=rev Log: Remove class Language - the only thing it was actually being used for is provided by LanguageRuntime already Removed: lldb/trunk/include/lldb/Core/Language.h

[Lldb-commits] [lldb] r246172 - More cleanup to make sure no one plays with DWARFDebugInfoEntry. Clients outside of DWARFDebugInfoEntry of DWARFCompileUnit should use DWARFDIE only.

2015-08-27 Thread Greg Clayton via lldb-commits
Author: gclayton Date: Thu Aug 27 13:09:44 2015 New Revision: 246172 URL: http://llvm.org/viewvc/llvm-project?rev=246172&view=rev Log: More cleanup to make sure no one plays with DWARFDebugInfoEntry. Clients outside of DWARFDebugInfoEntry of DWARFCompileUnit should use DWARFDIE only. Modified:

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread Zachary Turner via lldb-commits
zturner added a subscriber: zturner. zturner added a comment. This is definitely going to be a serious problem once I integrate our PDB support for Windows http://reviews.llvm.org/rL245090 ___ lldb-commits mailing list lldb-commits@lists.llvm.org ht

Re: [Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

2015-08-27 Thread Zachary Turner via lldb-commits
This is definitely going to be a serious problem once I integrate our PDB support for Windows On Wed, Aug 26, 2015 at 9:09 PM Dawn Perchik via lldb-commits < lldb-commits@lists.llvm.org> wrote: > dawn added a subscriber: dawn. > dawn added a comment. > > Hi Greg, > > While I really appreciate tha

Re: [Lldb-commits] [PATCH] D12356: [MIPS64] Emulate MSA branch instructions

2015-08-27 Thread Tamas Berghammer via lldb-commits
tberghammer added a comment. I don't know too much about mips so I haven't checked if the emulation is actually correct but the general concept looks good to me. I added a few comments inline but they are mostly suggestions what you might want to consider. Comment at: source/

Re: [Lldb-commits] [PATCH] D12356: [MIPS64] Emulate MSA branch instructions

2015-08-27 Thread Sagar Thakur via lldb-commits
sagar updated this revision to Diff 33323. sagar added a comment. Addressed review comments Repository: rL LLVM http://reviews.llvm.org/D12356 Files: source/Plugins/Instruction/MIPS64/EmulateInstructionMIPS64.cpp source/Plugins/Instruction/MIPS64/EmulateInstructionMIPS64.h Index: source