[Lldb-commits] [PATCH] D128264: [lldb/dyld-posix] Avoid reading the module list in inconsistent states

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Jun 23 01:47:59 PDT 2022


labath marked an inline comment as done.
labath added inline comments.


================
Comment at: lldb/source/Plugins/DynamicLoader/POSIX-DYLD/DynamicLoaderPOSIXDYLD.cpp:441
+              &m_process->GetTarget()) == m_interpreter_base) {
+        ModuleSP interpreter_sp = m_interpreter_module.lock();
+        if (m_interpreter_module.lock() == nullptr) {
----------------
emrekultursay wrote:
> Use this variable below, instead of calling lock() again?
Yes. In fact, that's why I created that variable in the first place. :)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128264/new/

https://reviews.llvm.org/D128264



More information about the lldb-commits mailing list