Author: jingham
Date: Wed May 24 21:24:18 2017
New Revision: 303832
URL: http://llvm.org/viewvc/llvm-project?rev=303832&view=rev
Log:
Fix the warning when you pass -c to step/next/si/ni.
During some cleanup the test for whether the thread plan
accepted an iteration count was reversed, so we give
Author: kamil
Date: Wed May 24 18:59:50 2017
New Revision: 303823
URL: http://llvm.org/viewvc/llvm-project?rev=303823&view=rev
Log:
Correct compiler warnings and Debug build of the NetBSD target
Correct files present only in the NetBSD build.
Modified:
lldb/trunk/source/Plugins/Process/NetBS
jmajors updated this revision to Diff 100178.
jmajors added a comment.
Format changes.
https://reviews.llvm.org/D32930
Files:
unittests/CMakeLists.txt
unittests/tools/CMakeLists.txt
unittests/tools/lldb-server/CMakeLists.txt
unittests/tools/lldb-server/inferior/thread_inferior.cpp
uni
jmajors updated this revision to Diff 100151.
jmajors added a comment.
Added m_ prefix to member variables.
Converted lldb-server start to single thread.
https://reviews.llvm.org/D32930
Files:
unittests/CMakeLists.txt
unittests/tools/CMakeLists.txt
unittests/tools/lldb-server/CMakeLists.t
jingham accepted this revision.
jingham added a comment.
This revision is now accepted and ready to land.
The fix seems good.
The fact that a breakpoint hit while evaluating an expression doesn't check the
condition is a known limitation.
You've got to protect against artificial recursions in h
Formatting is done.
Created D33504, and it is accepted now, but what should I do next (I haven’t
commit access)?
Thanks,
Tatyana
From: Abid, Hafiz [mailto:hafiz_a...@mentor.com]
Sent: Wednesday, 24 May, 2017 12:37 PM
To: Tatyana Krasnukha ;
lldb-commits@lists.llvm.org
Subject: Re: [Lldb-commits
jafl added a comment.
Thanks for the suggestions. I will get around to this - just swamped right now!
Repository:
rL LLVM
https://reviews.llvm.org/D32366
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mai
labath added inline comments.
Comment at:
packages/Python/lldbsuite/test/python_api/sbstructureddata/TestStructuredDataAPI.py:43
+# Now launch the process, and do not stop at entry point.
+process = target.LaunchSimple(
+None, None, self.get_process_w
clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.
No need for the "StructuredDataType::" or "lldb::StructuredDataType::"
prefixes. Since we aren't using an enum class (it was before) and since all
enumerators start with "eStruct
clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.
If all R"( strings are under 80 characters, this is good to go. If not, just
fix and submit, no extra review needed.
https://reviews.llvm.org/D32585
___
abhishek.aggarwal added a comment.
My comments are inlined. Please let me know if something still needs to be
changed.
Comment at: include/lldb/API/SBStructuredData.h:60
+ //--
+ size_t GetSize() const;
+
--
abhishek.aggarwal updated this revision to Diff 100108.
abhishek.aggarwal marked 9 inline comments as done.
abhishek.aggarwal added a comment.
Updating https://reviews.llvm.org/D33434: Added new API to SBStructuredData
class
- Made changes according to feedback
https://reviews.llvm.org/D33434
labath requested review of this revision.
labath added a comment.
Let me know what you think of the fix, and please confirm whether the ignoring
of the breakpoint condition is a bug.
thanks.
https://reviews.llvm.org/D33283
___
lldb-commits mailing
labath updated this revision to Diff 100099.
labath added a comment.
Fixed version.
The original patch caused a regression in TestLoadUnload, which has only showed
up when running the remote test suite. The problem there was that we interrupted
the target just as it has hit the rendezvous breakpo
labath reopened this revision.
labath added a comment.
This revision is now accepted and ready to land.
Reopening for a re-review of a fix.
Repository:
rL LLVM
https://reviews.llvm.org/D33283
___
lldb-commits mailing list
lldb-commits@lists.llvm.
krytarowski added a comment.
In https://reviews.llvm.org/D32149#763008, @labath wrote:
> What was your decision on the core files? I was under the impression you were
> gonna add the zip files as well. If so, then they should go in at the same
> time.
In the same commit? If so I will try to a
Author: labath
Date: Wed May 24 06:57:28 2017
New Revision: 303740
URL: http://llvm.org/viewvc/llvm-project?rev=303740&view=rev
Log:
Revert "RunThreadPlan: Fix halting logic in IgnoreBreakpoints = false"
This reverts commit r303732, as it introduces a regression in
TestLoadUnload on android.
Mod
labath added a comment.
that sounds like an excellent idea, as it will check all executed commands, and
not the ones we've remembered checking. It should probably be an `lldbassert`
though. (And we'd need to check that the existing tests still pass after that.)
Repository:
rL LLVM
https://r
This revision was automatically updated to reflect the committed changes.
Closed by commit rL303732: RunThreadPlan: Fix halting logic in
IgnoreBreakpoints = false (authored by labath).
Changed prior to commit:
https://reviews.llvm.org/D33283?vs=99435&id=100058#toc
Repository:
rL LLVM
https:
Author: labath
Date: Wed May 24 04:46:48 2017
New Revision: 303732
URL: http://llvm.org/viewvc/llvm-project?rev=303732&view=rev
Log:
RunThreadPlan: Fix halting logic in IgnoreBreakpoints = false
Summary:
The function had logic to handle the case when the expression terminated
while we were trying
labath accepted this revision.
labath added a comment.
Looks good as far as I am concerned (please wait for greg's ok though)
https://reviews.llvm.org/D32585
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mai
labath added a comment.
Thanks for writing the test. We just need to make sure it runs in a stable
manner.
Comment at:
packages/Python/lldbsuite/test/functionalities/thread/num_threads/TestNumThreads.py:91
+# thread3 function. All of these threads should show as one s
Please put it as review in Phabricator so others can comment too. I am
wondering how we can test this behaviour.
One can use -fno-asynchronous-unwind-tables with clang to disable generation of
.eh_frame but the startup code
still brings in that section.
You also need to run clang-format to ma
labath added a comment.
What was your decision on the core files? I was under the impression you were
gonna add the zip files as well. If so, then they should go in at the same time.
Repository:
rL LLVM
https://reviews.llvm.org/D32149
___
lldb-c
labath added a comment.
thank you
https://reviews.llvm.org/D33283
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
ravitheja updated this revision to Diff 100051.
ravitheja added a comment.
Running clang-format.
https://reviews.llvm.org/D32585
Files:
docs/lldb-gdb-remote.txt
include/lldb/API/SBTrace.h
include/lldb/Core/TraceOptions.h
include/lldb/Host/common/NativeProcessProtocol.h
include/lldb/Ta
bgianfo updated this revision to Diff 100047.
bgianfo marked an inline comment as done.
bgianfo added a comment.
Address Pavel/Jim/Greg's feedback with the addition of a new test.
I followed Jim's advice and extended the existing num_threads suite
so that we start more thread3's. The new test_uni
27 matches
Mail list logo
