diff options
| author | Adrian McCarthy <amccarth@google.com> | 2017-09-07 18:29:48 +0000 |
|---|---|---|
| committer | Adrian McCarthy <amccarth@google.com> | 2017-09-07 18:29:48 +0000 |
| commit | 8fe23bc5200b7963deaa6ab89fa4a54b81f6fd47 (patch) | |
| tree | b8d6017e81764e2bfd073aa9d37711cf9cedbdde /lldb/source/Plugins/Process/minidump/ThreadMinidump.h | |
| parent | 8af4e23d1ee7e93e9bda835be2b97550a13c0737 (diff) | |
| download | bcm5719-llvm-8fe23bc5200b7963deaa6ab89fa4a54b81f6fd47.tar.gz bcm5719-llvm-8fe23bc5200b7963deaa6ab89fa4a54b81f6fd47.zip | |
Fix for bug 34510 - Minidump target does not resolve new symbols correctly
Even though the content of the minidump does not change in a debugging session,
frames can't be indiscriminately be cached since modules and symbols can be
explicitly added after the minidump is loaded.
The fix is simple, just let the base Thread::ClearStackFrames() do its job.
submitted by amccarth on behalf of lemo
Bug: https://bugs.llvm.org/show_bug.cgi?id=34510
Differential Revision: https://reviews.llvm.org/D37527
llvm-svn: 312735
Diffstat (limited to 'lldb/source/Plugins/Process/minidump/ThreadMinidump.h')
| -rw-r--r-- | lldb/source/Plugins/Process/minidump/ThreadMinidump.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/lldb/source/Plugins/Process/minidump/ThreadMinidump.h b/lldb/source/Plugins/Process/minidump/ThreadMinidump.h index 97db452edff..74ac44f74dc 100644 --- a/lldb/source/Plugins/Process/minidump/ThreadMinidump.h +++ b/lldb/source/Plugins/Process/minidump/ThreadMinidump.h @@ -37,8 +37,6 @@ public: lldb::RegisterContextSP CreateRegisterContextForFrame(StackFrame *frame) override; - void ClearStackFrames() override; - protected: lldb::RegisterContextSP m_thread_reg_ctx_sp; llvm::ArrayRef<uint8_t> m_gpregset_data; |
