<div dir="ltr">This looks fine in general. I'd like some testcases though please.<div><br></div><div>Thanks!</div><div><br></div><div>-eric</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 21, 2013 at 3:37 PM, Kaylor, Andrew <span dir="ltr"><<a href="mailto:andrew.kaylor@intel.com" target="_blank">andrew.kaylor@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="blue" vlink="purple">
<div>
<p class="MsoNormal">This patch adds a new method, getLineInfoForAddressRange, to the DIContext interface class and provides an implementation for that method in the DebugInfo code.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">This is a subset of a patch I submitted last week (<a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130114/162210.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130114/162210.html</a>).  There
 is nothing new in this patch, I have simply isolated the DebugInfo-specific changes to make review easier.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">The new method will be used by the IntelJITEventListener to provide line information for profiling code generated by MCJIT, but these changes are not dependent on the JIT event listener changes.  I’ll resubmit the JIT event listener changes
 momentarily for context.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">-Andy<u></u><u></u></p>
</div>
</div>

</blockquote></div><br></div>