[PATCH] Remove leftover profiling pieces

Chandler Carruth chandlerc at google.com
Mon Oct 14 08:06:43 PDT 2013


LGTM
On Oct 10, 2013 12:46 PM, "Alastair Murray" <alastairmurray42 at gmail.com>
wrote:

> Hi all,
>
> 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.
>
> (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.)
>
> Patches:
> 1-remove-profiling-utils.patch
> 'svn remove lib/Transforms/**Instrumentation/**ProfilingUtils.*', and a
> few updates that this required.
>
> 2-svn-remove-utils-profile-pl.**patch
> 'svn remove utils/profile.pl'.  I'm pretty sure this hasn't worked in
> years, it definitely doesn't work now (it uses the removed opt arguments).
>
> 3-svn-remove-testsuite-**profile-makefile.patch
> 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.
>
> Tested by recompiling and 'make check-all', and 'make TEST=simple report'
> on the test-suite.
>
> Regards,
> Alastair.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131014/8d73c57c/attachment.html>


More information about the llvm-commits mailing list