lawrence_danna created this revision.
lawrence_danna added reviewers: JDevlieghere, jasonmolenda, labath, zturner.
Herald added a project: LLDB.
This patch eliminates a bunch of boilerplate from
PythonDataObjects, as well as the use of virtual methods.
In my opinion it also makes the Reset logic
mstorsjo updated this revision to Diff 224757.
mstorsjo retitled this revision from "[LLDB] [Windows] Initial support for
ARM64 debugging" to "[LLDB] [Windows] Initial support for ARM64 register
contexts".
mstorsjo edited the summary of this revision.
mstorsjo added a reviewer: aleksandr.urakov.
llunak updated this revision to Diff 224739.
llunak edited the summary of this revision.
llunak added a comment.
Updated misleading description.
Repository:
rLLDB LLDB
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68910/new/
https://reviews.llvm.org/D68910
Files:
lldb/scripts/CMak
mgorny requested changes to this revision.
mgorny added a comment.
This revision now requires changes to proceed.
No, it isn't. Some distros patch some Python versions for that but it's not a
rule.
Repository:
rLLDB LLDB
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68910/new/
https
llunak updated this revision to Diff 224737.
llunak edited the summary of this revision.
llunak added a comment.
Added a unittest.
Also, the other discussed changes are at https://reviews.llvm.org/D68908 and
https://reviews.llvm.org/D68909 .
CHANGES SINCE LAST ACTION
https://reviews.llvm.org
llunak created this revision.
llunak added reviewers: clayborg, labath.
llunak added a project: LLDB.
Herald added subscribers: lldb-commits, JDevlieghere, mgorny.
On 64bit linux python is in lib64/python.
Repository:
rLLDB LLDB
https://reviews.llvm.org/D68910
Files:
lldb/scripts/CMakeList
llunak created this revision.
llunak added a reviewer: clayborg.
llunak added a project: LLDB.
Herald added subscribers: lldb-commits, JDevlieghere.
This makes it consistent with gdb tui, where 'f' is 'finish'.
See the discussion at https://reviews.llvm.org/D68541 .
Repository:
rLLDB LLDB
htt
llunak created this revision.
llunak added a reviewer: clayborg.
llunak added a project: LLDB.
Herald added subscribers: lldb-commits, JDevlieghere.
'd' would be much better used for up/down shortcuts, and this also removes
the possibility of ruining the whole debugging session by accidentally
grimar added inline comments.
Comment at: llvm/include/llvm/ObjectYAML/MinidumpYAML.h:113-114
+ ExceptionStream()
+ : Stream(StreamKind::Exception, minidump::StreamType::Exception) {
+memset(&MDExceptionStream, 0, sizeof(minidump::ExceptionStream));
+ }
---
tatyana-krasnukha added inline comments.
Comment at: lldb/CMakeLists.txt:228
+function(create_relative_symlink target dest_file output_dir output_name)
+ get_filename_component(dest_file ${dest_file} ABSOLUTE)
+ get_filename_component(output_dir ${output_dir} ABSOLU
10 matches
Mail list logo