Fix for bug 34510 - Minidump target does not resolve new symbols correctly

[lemo]

Summary

Even though the content of the minidump does no 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 it's job.

Bug: https://bugs.llvm.org/show_bug.cgi?id=34510

Differential Revision: https://reviews.llvm.org/D37527

[amccarth]

This looks fine.

Make sure you've run the LLDB tests. I don't expect this change to have much risk for breaking any of them, but it's a good habit. ninja check-lldb

For future patch reviews, please create the diffs with full context (e.g., git diff -U999999) and from the root of your source tree so that Phabricator shows paths rather than just file names.

If you're not using arc, make sure you amend your commit so that the description has a line like this:

`Differential Revision: https://reviews.llvm.org/D37527`

Capitalization and spacing are sensitive in those lines. When you push your change, that'll associate the revision with the review and close the review on the Phabricator side.

You can also mention the bug number in the description.

[dvlahovski]

LGTM

[lemo]

Thanks everyone. I don't have commit permissions, can someone please commit this on my behalf?

[amccarth]

I'll commit it momentarily.

lldb/trunk/source/Plugins/Process/minidump/ThreadMinidump.h

public:
 
   void RefreshStateAfterStop() override;
 
   lldb::RegisterContextSP GetRegisterContext() override;
 
   lldb::RegisterContextSP
   CreateRegisterContextForFrame(StackFrame *frame) override;
 
-  void ClearStackFrames() override;
-
 protected:
   lldb::RegisterContextSP m_thread_reg_ctx_sp;
   llvm::ArrayRef<uint8_t> m_gpregset_data;
 
   bool CalculateStopInfo() override;
 };
 
 } // namespace minidump
 } // namespace lldb_private
 
 #endif // liblldb_ThreadMinidump_h_

lldb/trunk/source/Plugins/Process/minidump/ThreadMinidump.cpp

ThreadMinidump::ThreadMinidump(Process &process, const MinidumpThread &td,
                                llvm::ArrayRef<uint8_t> gpregset_data)
     : Thread(process, td.thread_id), m_thread_reg_ctx_sp(),
       m_gpregset_data(gpregset_data) {}
 
 ThreadMinidump::~ThreadMinidump() {}
 
 void ThreadMinidump::RefreshStateAfterStop() {}
 
-void ThreadMinidump::ClearStackFrames() {}
-
 RegisterContextSP ThreadMinidump::GetRegisterContext() {
   if (!m_reg_context_sp) {
     m_reg_context_sp = CreateRegisterContextForFrame(nullptr);
   }
   return m_reg_context_sp;
 }
 
 RegisterContextSP