<p dir="ltr">LGTM</p>
<div class="gmail_quote">On Oct 10, 2013 12:46 PM, "Alastair Murray" <<a href="mailto:alastairmurray42@gmail.com">alastairmurray42@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi all,<br>
<br>
Further to Chandler Carruth's commit r191835 to remove the old profiling framework, here are few more patches to remove a few things that I think he missed.  Chandler, I've CC'ed you in case these were left deliberately.<br>

<br>
(As an aside, as someone who has dealt with LLVM profiling before and who was responsible for the most recent abandoned attempt to reboot it, I'm very happy to see a clean slate and I hope to be able to provide patches building on the newly proposed models.)<br>

<br>
Patches:<br>
1-remove-profiling-utils.patch<br>
'svn remove lib/Transforms/<u></u>Instrumentation/<u></u>ProfilingUtils.*', and a few updates that this required.<br>
<br>
2-svn-remove-utils-profile-pl.<u></u>patch<br>
'svn remove utils/<a href="http://profile.pl" target="_blank">profile.pl</a>'.  I'm pretty sure this hasn't worked in years, it definitely doesn't work now (it uses the removed opt arguments).<br>
<br>
3-svn-remove-testsuite-<u></u>profile-makefile.patch<br>
In the test-suite, 'svn remove TEST.profile.*', I'd be happy to revive these in the future, but for now they use the removed opt arguments.<br>
<br>
Tested by recompiling and 'make check-all', and 'make TEST=simple report' on the test-suite.<br>
<br>
Regards,<br>
Alastair.<br>
</blockquote></div>