[Lldb-commits] [PATCH] D55399: If we still have all_image_infos use it in DynamicLoaderMacOSDYLD to detect exec's

[Jim Ingham via Phabricator via lldb-commits]

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


================
Comment at: source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOS.cpp:104
+        if (image_infos_address != m_maybe_image_infos_address)
+          did_exec = true;
+      }
----------------
jingham wrote:
> jasonmolenda wrote:
> > Do you want to copy the image_infos_address value into m_maybe_image_infos_address so we can detect the next exec?
> You don't need to. If we've exec'ed we have to call DoInitialImageFetch again - since the world has changed - which resets the value.  I don't want to do it twice, but I can put in a comment explaining why if that would help.
Eh, that's dumb, it's just a single assign, so why not.  I added that in the committed version.


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D55399