<div dir="ltr">Okey dokey - I know one of the things we did (& I don't know when it happened compared to this change) is emit a hard list of variables onto any subprogram for an optimized (non -O0) function. So we never lose variables due to optimizations, or at least that's the intent.<br><br>As for D18477, I'm not sure why collectDeadVariables became dead with your change - I imagine we still want to stuff dead variables into the subprograms we did create (the ones that weren't /entirely/ optimized away (ones with remaining inlined instances, etc)) - but maybe we do that elsewhere?</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 5, 2016 at 3:14 PM, Adrian Prantl via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aprantl abandoned this revision.<br>
aprantl added a comment.<br>
<br>
Meanwhile I made an experiment and compiled XNU with <a href="http://reviews.llvm.org/D18477" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18477</a> and compared the debug info from before and after and found no missing types at all. It is plausible that r107027 was added to work around the fact that six years ago LLVM was a lot worse at preserving debug info for local variables. It is also possible that the call graph of the kernel changed in the mean time or a combination of both.<br>
<span class="im HOEnZb"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D18565" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18565</a><br>
<br>
<br>
<br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>