enlight added inline comments.
Comment at: tools/lldb-mi/MICmnLLDBUtilSBValue.cpp:21-22
+static const char *kUnknownValue = "??";
+static const char *kCompositeValuePlaceholder = "{...}";
+
ki.stfu wrote:
> use unnamed namespace here
I would like to use an anon
ki.stfu requested changes to this revision.
ki.stfu added inline comments.
This revision now requires changes to proceed.
Comment at: tools/lldb-mi/MICmnLLDBUtilSBValue.cpp:21-22
+static const char *kUnknownValue = "??";
+static const char *kCompositeValuePlaceholder = "{...}";
enlight created this revision.
enlight added reviewers: ki.stfu, abidh.
enlight added a subscriber: LLDB.
enlight set the repository for this revision to rL LLVM.
Placeholder c-strings don't need to be instance variables.
Repository:
rL LLVM
https://reviews.llvm.org/D25592
Files:
tools/lld
Author: jingham
Date: Thu Oct 13 20:11:19 2016
New Revision: 284183
URL: http://llvm.org/viewvc/llvm-project?rev=284183&view=rev
Log:
This test passes on i386 now.
Modified:
lldb/trunk/packages/Python/lldbsuite/test/lang/objc/ivar-IMP/TestObjCiVarIMP.py
Modified:
lldb/trunk/packages/Python
Author: jingham
Date: Thu Oct 13 20:03:03 2016
New Revision: 284182
URL: http://llvm.org/viewvc/llvm-project?rev=284182&view=rev
Log:
This test is passing on i386 now.
Modified:
lldb/trunk/packages/Python/lldbsuite/test/lang/objc/objc-class-method/TestObjCClassMethod.py
Modified:
lldb/trun
tfiala requested changes to this revision.
tfiala added a comment.
This revision now requires changes to proceed.
Okay. I think we need a minor tweak, to drop the REQUIRED on the Backtrace
usage.
https://reviews.llvm.org/D23977
___
lldb-commits ma
tfiala accepted this revision.
tfiala added a comment.
This revision is now accepted and ready to land.
LGTM.
https://reviews.llvm.org/D25570
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/ll
beanz added a comment.
Worth noting, the CMake bug now has a fix in a pull request:
https://gitlab.kitware.com/cmake/cmake/merge_requests/172
https://reviews.llvm.org/D25570
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llv
dvlahovski updated this revision to Diff 74551.
dvlahovski added a comment.
Forgot to run clang-format. Also changed a helper function in the tests to make
it simpler.
https://reviews.llvm.org/D25569
Files:
source/Plugins/Process/minidump/MinidumpParser.cpp
source/Plugins/Process/minidump/
beanz created this revision.
beanz added reviewers: tfiala, zturner, spyffe.
beanz added a subscriber: lldb-commits.
Herald added a subscriber: mgorny.
This patch adds support for installing public headers in LLDB.framework, and
symlinking the headers into the build directory.
While writing the
dvlahovski created this revision.
dvlahovski added reviewers: labath, zturner.
dvlahovski added subscribers: amccarth, lldb-commits.
Herald added subscribers: modocache, mgorny, beanz.
Now the Minidump parser can parse the:
1. MemoryInfoList - containing region info about memory ranges (readable,
tfiala added a comment.
(Retro) Ah nice, I like the generator expression you put in there.
Repository:
rL LLVM
https://reviews.llvm.org/D25490
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinf
tfiala added a comment.
(Retro) LGTM.
Repository:
rL LLVM
https://reviews.llvm.org/D25489
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
tfiala added a comment.
(Retro) LGTM.
Repository:
rL LLVM
https://reviews.llvm.org/D25488
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
tfiala added a comment.
(Retro) yep, this looked fine.
Repository:
rL LLVM
https://reviews.llvm.org/D25487
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
In case of Linux and Android we are using the qRegisterInfo packet and
lldb-server fills it in based on the register definitions inside LLDB so
for those targets it would be important to have all of the alias registers
available.
I don't have an AArch64-BE target at hand but I am pretty sure you a
Author: rnchamberlain
Date: Thu Oct 13 07:11:00 2016
New Revision: 284114
URL: http://llvm.org/viewvc/llvm-project?rev=284114&view=rev
Log:
[lldb] Improve identification of Linux core dumps. Fix for bug #30485.
Summary:
ObjectFileELF::RefineModuleDetailsFromNote() identifies Linux core dumps by
17 matches
Mail list logo
