[Lldb-commits] [PATCH] D141702: [lldb/crashlog] Make module loading use Scripted Process affordance

[Med Ismail Bennani via Phabricator via lldb-commits]

mib added inline comments.


================
Comment at: lldb/source/Plugins/Process/scripted/ScriptedProcess.cpp:493
 
   if (!loaded_images_sp->ForEach(reload_image))
     return ScriptedInterface::ErrorWithMessage<StructuredData::ObjectSP>(
----------------
I'm just thinking about this but I don't see any reason not to run this loop in parallel to accelerate dependency loading ... That could potentially mess up the progress reporting but I think in parallel it might not even be needed. @JDevlieghere Any objections ?


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

https://reviews.llvm.org/D141702