<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 25, 2017 at 5:03 PM, Mehdi AMINI 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">mehdi_amini added a comment.<br>
<span class="gmail-"><br>
In <a href="https://reviews.llvm.org/D33513#764629" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D33513#764629</a>, @ncharlie wrote:<br>
<br>
> In <a href="https://reviews.llvm.org/D33513#763708" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D33513#763708</a>, @davide wrote:<br>
><br>
> > Thanks for your first contribution, great GSoC start!<br>
><br>
><br>
> Thanks! This was a good chance to look closely at the code to start learning how it works.<br>
><br>
> > One note: this manifests with ThinLTO, but it might be a more generic metadata issue. Are you able to reproduce without ThinLTO  (and maybe with a single file?)<br>
><br>
> The bug only happens when lazy loading Metadata. Lazy loading occurs when module level metadata is being imported by ThinLTO (see <a href="https://reviews.llvm.org/diffusion/L/browse/llvm/trunk/lib/Bitcode/Reader/MetadataLoader.cpp;303885$781" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>diffusion/L/browse/llvm/trunk/<wbr>lib/Bitcode/Reader/<wbr>MetadataLoader.cpp;303885$781</a>)<wbr>. I believe this is the only path that leads to lazy loading, so I don't think it can happen elsewhere.<br>
<br>
<br>
</span>If there is no other path to exercise LazyLoading of Metadata, let's just create one! (add an option to llvm-dis for example).<br></blockquote><div><br></div><div>I believe that there already is one, -materialize-metadata.</div></div>-- <br><div class="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div></div>