<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div style>Hi Andy,</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div lang="EN-US" link="blue" vlink="purple"><div><div>This code is a bit tricky to test because the changes don’t modify the behavior of any of the standard llvm tools.  I have been testing in with the llvm-jitlistner tool, but that only gets built if Intel JIT profiling is enabled in the build, which it isn’t for any of the buildbots.</div>
<div></div></div></div></blockquote><div><br></div><div style>Right.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div lang="EN-US" link="blue" vlink="purple"><div><div>Is there a better way to do this?</div><div> </div></div></div></blockquote><div><br></div><div style>Maybe using llvm-rtdyld to load a pre-generated .o file where you know an address and then use that to look for something? You'd have to add an option to llvm-rtdyld to print some line information/debug information for a given .o file but it'd probably be a good way to test?</div>
<div style><br></div><div style>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple">
<div><div>-Andy</div>
<p class=""><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p>
<div style="border-style:solid none none;border-top-color:rgb(181,196,223);border-top-width:1pt;padding:3pt 0in 0in">
<p class=""><b><span style="font-size:10pt;font-family:Tahoma,sans-serif">From:</span></b><span style="font-size:10pt;font-family:Tahoma,sans-serif"> Eric Christopher [mailto:<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>]
<br>
<b>Sent:</b> Thursday, January 24, 2013 1:03 AM<br>
<b>To:</b> Kaylor, Andrew<br>
<b>Cc:</b> llvm-commits<br>
<b>Subject:</b> Re: [PATCH] [Review request] Adding getLineInforForAddressRange() to DIContext<u></u><u></u></span></p>
</div><div><div class="h5">
<p class=""><u></u> <u></u></p>
<div>
<p class="">This looks fine in general. I'd like some testcases though please.<u></u><u></u></p>
<div>
<p class=""><u></u> <u></u></p>
</div>
<div>
<p class="">Thanks!<u></u><u></u></p>
</div>
<div>
<p class=""><u></u> <u></u></p>
</div>
<div>
<p class="">-eric<u></u><u></u></p>
</div>
</div>
<div>
<p class="" style="margin-bottom:12pt"><u></u> <u></u></p>
<div>
<p class="">On Mon, Jan 21, 2013 at 3:37 PM, Kaylor, Andrew <<a href="mailto:andrew.kaylor@intel.com" target="_blank">andrew.kaylor@intel.com</a>> wrote:<u></u><u></u></p>
<div>
<div>
<p class="">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=""> <u></u><u></u></p>
<p class="">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=""> <u></u><u></u></p>
<p class="">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=""> <u></u><u></u></p>
<p class="">-Andy<u></u><u></u></p>
</div>
</div>
</div>
<p class=""><u></u> <u></u></p>
</div>
</div></div></div>
</div>

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