[PATCH] D18468: Profile summary cleanup.

Easwaran Raman via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 25 14:36:50 PDT 2016


eraman updated this revision to Diff 51686.
eraman added a comment.

My original change had a bug due to the fact that sample profile head samples are also counted in body samples (I couldn't run the tests yesterday since the head was broken). I had to undo some of the refactoring from my original patch, but the main stuff - moving max function count and num functions to the base class - still remains the same. I have tested this.


http://reviews.llvm.org/D18468

Files:
  include/llvm/ProfileData/ProfileCommon.h
  lib/ProfileData/ProfileSummary.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18468.51686.patch
Type: text/x-patch
Size: 6949 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160325/d8ef1a80/attachment.bin>


More information about the llvm-commits mailing list