<div dir="ltr"><br><div>LGTM. Please commit.</div><div><br></div><div>Thanks,</div><div>Manman</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Oct 8, 2013 at 11:30 PM, Mishne, Alon <span dir="ltr"><<a href="mailto:alon.mishne@intel.com" target="_blank">alon.mishne@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="#0563C1" vlink="#954F72">
<div>
<p class="MsoNormal">When processing a module, DebugInfoFinder only goes over the first compile unit in the module's debug metadata, instead of going over all the compile units; there's even a "FIXME" comment there about this issue. I've opened a bug report
 about it at <a href="http://llvm.org/bugs/show_bug.cgi?id=17507" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=17507</a><u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Attached is a patch which fixes this and adds a test to verify that more than one compile unit can be processed.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p><u></u><span>-<span style="font:7.0pt "Times New Roman"">       
</span></span><u></u><span dir="LTR"></span>Alon<u></u><u></u></p>
</div>
<p>---------------------------------------------------------------------<br>
Intel Israel (74) Limited</p>

<p>This e-mail and any attachments may contain confidential material for<br>
the sole use of the intended recipient(s). Any review or distribution<br>
by others is strictly prohibited. If you are not the intended<br>
recipient, please contact the sender and delete all copies.</p></div>

<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>