aprantl accepted this revision. aprantl added inline comments. This revision is now accepted and ready to land.
================ Comment at: lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp:1118 +bool DynamicLoaderMacOSXDYLD::IsSafeToCallAPI() { + if (ReadAllImageInfosStructure()) + return m_dyld_all_image_infos.libSystemInitialized; ---------------- Out of curiosity: Is this an error when ReadAllImageInfosStructure() returns false? In that case a better API might be one that also bubbles up the error. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99314/new/ https://reviews.llvm.org/D99314 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits