<div dir="ltr">It's an interesting idea, thanks! I don't object moving code around if there's a strong case for it, but I'd like to keep the fix small and simple for now, but it's worth considering if the current minidump loading path will need more flexibility.<div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 23, 2018 at 1:27 PM, Greg Clayton via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg added a comment.<br>
<span class=""><br>
In <a href="https://reviews.llvm.org/D51176#1211433" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D51176#1211433</a>, @jingham wrote:<br>
<br>
> The other option would be to move the code that populates the section load list into the mini dump dynamic loader.  That has the benefit of keeping this consistent with the other process plugins, but OTOH is just moving code around...<br>
<br>
<br>
</span>Yes the dynamic loader plug-ins aren't hard to write and the code already exists in the ProcessMinidump. Then you would request the plug-in by name instead of passing a nullptr as the name in ProcessMinidump::<wbr>GetDynamicLoader().<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rLLDB LLDB<br>
<br>
<a href="https://reviews.llvm.org/D51176" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D51176</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>