[llvm-commits] [PATCH 2/4] Profile metadata: Profile loader

[Manman Ren]

Sorry for the delay, I will try to review the patches tomorrow.

Thanks,
Manman

On Aug 18, 2012, at 6:37 AM, Alastair Murray wrote:

> Hello all,
> 
> Please review the attached patch.
> 
> This is the second of four patches to support setting branch weight metadata by profiling.  It depends on the first patch.
> 
> There are also three binary profile data files attached that are used for testing, these should go in test/Analysis/Profiling.
> 
> This patch implements the profile loader which can load profile data generated by programs compiled with "-insert-edge-profiling".  It also provides tests and moves three function definitions out of ProfileInfo into the new profiling code.  The pass is invoked by "-profile-metadata-loader", but I believe that once ProfileInfo etc. is removed it should take over the "-profile-loader" switch that that currently uses.
> 
> The attached patch has been cleaned up a little from the version sent to llvm-dev last week, some class renaming, commenting etc.  It is largely the same in logical terms though.
> 
> It has been tested via the attached tests and by '-profile-metadata-loader -std-compile-opts' over test-suite (using the Makefile to come in patch #4).
> 
> Regards,
> Alastair.
> <weights_2_profiler_loader.patch><load-branch-weights-ifs.ll.prof_data><load-branch-weights-loops.ll.prof_data><load-branch-weights-switches.ll.prof_data>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits