[Lldb-commits] [PATCH] D89925: [lldb] Fix a regression introduced by D75730
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Oct 22 02:47:10 PDT 2020
labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.
If needed, we can add some `#ifdef`s to produce suitable instructions on other architectures too.
================
Comment at: lldb/source/Core/Disassembler.cpp:544
const char *plugin_name, const char *flavor,
const ExecutionContext &exe_ctx,
uint32_t num_instructions,
----------------
Maybe change this to take a `StackFrame&` argument? The only caller StackFrame::Disassemble, so the frame argument is always valid (and a lot of this defensive code is not needed).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89925/new/
https://reviews.llvm.org/D89925
More information about the lldb-commits
mailing list