[Lldb-commits] Buildbot numbers for the last week of 11/5/2017 - 11/11/2017

2017-11-13 Thread Galina Kistanova via lldb-commits
Hello everyone, Below are some buildbot numbers for the last week of 11/5/2017 - 11/11/2017. Please see the same data in attached csv files: The longest time each builder was red during the week; "Status change ratio" by active builder (percent of builds that changed the builder status from gree

[Lldb-commits] Buildbot numbers for the week of 10/29/2017 - 11/4/2017

2017-11-13 Thread Galina Kistanova via lldb-commits
Hello everyone, Below are some buildbot numbers for the week of 10/29/2017 - 11/4/2017. Please see the same data in attached csv files: The longest time each builder was red during the week; "Status change ratio" by active builder (percent of builds that changed the builder status from greed to

[Lldb-commits] Buildbot numbers for the week of 10/22/2017 - 10/28/2017

2017-11-13 Thread Galina Kistanova via lldb-commits
Hello everyone, Below are some buildbot numbers for the week of 10/22/2017 - 10/28/2017. Please see the same data in attached csv files: The longest time each builder was red during the week; "Status change ratio" by active builder (percent of builds that changed the builder status from greed to

[Lldb-commits] [PATCH] D39578: Fix a couple of self-assignments using memcpy.

2017-11-13 Thread Don Hinton via Phabricator via lldb-commits
hintonda updated this revision to Diff 122730. hintonda added a comment. - Address comments. https://reviews.llvm.org/D39578 Files: source/Core/RegisterValue.cpp source/Core/Value.cpp Index: source/Core/Value.cpp === --- sour

[Lldb-commits] [PATCH] D39966: Add a data formatter for libc++ std::bitset

2017-11-13 Thread Jim Ingham via Phabricator via lldb-commits
jingham accepted this revision. jingham added a comment. This revision is now accepted and ready to land. Looks fine. Seems like Update doesn't need to grab the bool type on every pass, but I don't think that matters IRL. Thanks for adding this. Comment at: source/Plugins/La

[Lldb-commits] [lldb] r318079 - Reformat a comment. NFC.

2017-11-13 Thread Stephane Sezer via lldb-commits
Author: sas Date: Mon Nov 13 12:13:25 2017 New Revision: 318079 URL: http://llvm.org/viewvc/llvm-project?rev=318079&view=rev Log: Reformat a comment. NFC. Modified: lldb/trunk/include/lldb/lldb-private-defines.h Modified: lldb/trunk/include/lldb/lldb-private-defines.h URL: http://llvm.org/v

[Lldb-commits] [PATCH] D39969: Set error status in ObjectFile::LoadInMemory if it is not set

2017-11-13 Thread Tatyana Krasnukha via Phabricator via lldb-commits
tatyana-krasnukha created this revision. Herald added a subscriber: ki.stfu. When breakpoints intersect a section, process->WriteMemory returns 0 without setting the error, LoadInMemory breaks execution and returns this error, but client code treat it as success. Just added setting of error sta

[Lldb-commits] [PATCH] D39967: Refactoring of MemoryWrite function

2017-11-13 Thread Tatyana Krasnukha via Phabricator via lldb-commits
tatyana-krasnukha added inline comments. Comment at: source/Target/Process.cpp:2462 - // We weren't able to write all of the requested bytes, we - // are done looping and will return the number of bytes that - // we have written so far.

[Lldb-commits] [PATCH] D39967: Refactoring of MemoryWrite function

2017-11-13 Thread Tatyana Krasnukha via Phabricator via lldb-commits
tatyana-krasnukha created this revision. Comments in this function didn't match code. Moved it in right place and grouped cases, that need only WriteMemoryPrivate call, at the top of the function. Applied clang-format. Repository: rL LLVM https://reviews.llvm.org/D39967 Files: source/Tar

[Lldb-commits] [PATCH] D39966: Add a data formatter for libc++ std::bitset

2017-11-13 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. Herald added a subscriber: mgorny. Herald added a reviewer: EricWF. https://reviews.llvm.org/D39966 Files: packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/bitset/Makefile packages/Python/lldbsuite/test/functionalities/dat

[Lldb-commits] [PATCH] D39578: Fix a couple of self-assignments using memcpy.

2017-11-13 Thread Greg Clayton via Phabricator via lldb-commits
clayborg requested changes to this revision. clayborg added a comment. This revision now requires changes to proceed. First change looks good. Second one we can probably avoid doing anything in Value::AppendDataToHostBuffer and return 0. No need to copy data over itself. Comme

Re: [Lldb-commits] [lldb] r318039 - Revert "[lldb] Use OrcMCJITReplacement rather than MCJIT as the underlying JIT for LLDB"

2017-11-13 Thread Lang Hames via lldb-commits
For the curious: The relocation issue is one of two known incompatibilities between OrcMCJIT and MCJIT. I'm working on an ORC refactor that should eliminate both, at which point we can try this out again (with OrcMCJITReplacement properly #included this time). Cheers, Lang. On Mon, Nov 13, 2017 a

Re: [Lldb-commits] [lldb] r318039 - Revert "[lldb] Use OrcMCJITReplacement rather than MCJIT as the underlying JIT for LLDB"

2017-11-13 Thread Lang Hames via lldb-commits
Oops -- I thought I'd reverted this ages ago. Evidently not. Thanks Pavel! -- Lang. On Mon, Nov 13, 2017 at 6:03 AM, Pavel Labath via lldb-commits < lldb-commits@lists.llvm.org> wrote: > Author: labath > Date: Mon Nov 13 06:03:17 2017 > New Revision: 318039 > > URL: http://llvm.org/viewvc/llvm-p

[Lldb-commits] [PATCH] D39578: Fix a couple of self-assignments using memcpy.

2017-11-13 Thread Don Hinton via Phabricator via lldb-commits
hintonda added a comment. ping? https://reviews.llvm.org/D39578 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] r318052 - Fix netbsd, freebsd and osx builds for ArchSpec move

2017-11-13 Thread Pavel Labath via lldb-commits
Author: labath Date: Mon Nov 13 08:47:37 2017 New Revision: 318052 URL: http://llvm.org/viewvc/llvm-project?rev=318052&view=rev Log: Fix netbsd, freebsd and osx builds for ArchSpec move Modified: lldb/trunk/include/lldb/Utility/ArchSpec.h lldb/trunk/source/Host/macosx/Host.mm lldb/tru

[Lldb-commits] [PATCH] D39896: Remove last Host usage from ArchSpec

2017-11-13 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL318046: Remove last Host usage from ArchSpec (authored by labath). Changed prior to commit: https://reviews.llvm.org/D39896?vs=122416&id=122651#toc Repository: rL LLVM https://reviews.llvm.org/D3989

[Lldb-commits] [lldb] r318046 - Remove last Host usage from ArchSpec

2017-11-13 Thread Pavel Labath via lldb-commits
Author: labath Date: Mon Nov 13 07:57:20 2017 New Revision: 318046 URL: http://llvm.org/viewvc/llvm-project?rev=318046&view=rev Log: Remove last Host usage from ArchSpec Summary: In D39387, I was quick to jump to conclusion that ArchSpec has no external dependencies. It turns there still was one

[Lldb-commits] [PATCH] D39896: Remove last Host usage from ArchSpec

2017-11-13 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. If I understand correctly, everyone agrees with this change, so I will commit it soon(ish). Below are my thoughs on some of the things said in this thread. @zturner wrote: > Super awesome. When you do move it to Utility, can you run the deps python > script to see if an

[Lldb-commits] [PATCH] D39844: CompilerType: Add ability to retrieve an integral template argument

2017-11-13 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL318040: CompilerType: Add ability to retrieve an integral template argument (authored by labath). Repository: rL LLVM https://reviews.llvm.org/D39844 Files: lldb/trunk/include/lldb/Symbol/ClangASTCo

[Lldb-commits] [lldb] r318040 - CompilerType: Add ability to retrieve an integral template argument

2017-11-13 Thread Pavel Labath via lldb-commits
Author: labath Date: Mon Nov 13 06:26:21 2017 New Revision: 318040 URL: http://llvm.org/viewvc/llvm-project?rev=318040&view=rev Log: CompilerType: Add ability to retrieve an integral template argument Summary: Despite it's name, GetTemplateArgument was only really working for Type template argume

[Lldb-commits] [lldb] r318039 - Revert "[lldb] Use OrcMCJITReplacement rather than MCJIT as the underlying JIT for LLDB"

2017-11-13 Thread Pavel Labath via lldb-commits
Author: labath Date: Mon Nov 13 06:03:17 2017 New Revision: 318039 URL: http://llvm.org/viewvc/llvm-project?rev=318039&view=rev Log: Revert "[lldb] Use OrcMCJITReplacement rather than MCJIT as the underlying JIT for LLDB" This commit really did not introduce any functional changes (for most peop