[all-commits] [llvm/llvm-project] d14d52: [lldb][test] Add extra logging for module cache test

[David Spickett via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d14d52158bc444e2d036067305cf54aeea7c9edb
      https://github.com/llvm/llvm-project/commit/d14d52158bc444e2d036067305cf54aeea7c9edb
  Author: David Spickett <david.spickett at linaro.org>
  Date:   2023-12-20 (Wed, 20 Dec 2023)

  Changed paths:
    M lldb/source/Target/Thread.cpp
    M lldb/test/API/python_api/global_module_cache/TestGlobalModuleCache.py

  Log Message:
  -----------
  [lldb][test] Add extra logging for module cache test

And remove the workaround I was trying, as this logging may prove what
the actual issue is.

Which I think is that the thread plan map in Process is cleared before
the threads are destroyed. So Thread::ShouldStop could be getting
the current plan, then the plan map is cleared, then Thread::ShouldStop
is deciding based on that plan to pop a plan from the now empty stack.