This is an archive of the discontinued LLVM Phabricator instance.

Sample profile summary cleanup
ClosedPublic

Authored by eraman on Mar 28 2016, 12:36 PM.

Details

Summary

Replace references to MaxHeadSamples with MaxFunctionCount

Diff Detail

Event Timeline

eraman updated this revision to Diff 51824.Mar 28 2016, 12:36 PM
eraman retitled this revision from to Sample profile summary cleanup.
eraman updated this object.
eraman added reviewers: dnovillo, danielcdh.
eraman added a subscriber: llvm-commits.
dnovillo accepted this revision.Mar 28 2016, 12:42 PM
dnovillo edited edge metadata.

LGTM

Nothing here seems to change the format of the external file, but could you double check with the AutoFDO converter that we can still read Perf profiles?

Thanks!

This revision is now accepted and ready to land.Mar 28 2016, 12:42 PM
danielcdh accepted this revision.Mar 28 2016, 12:43 PM
danielcdh edited edge metadata.

LGTM

Nothing here seems to change the format of the external file, but could you double check with the AutoFDO converter that we can still read Perf profiles?

Thanks!

Verified that this doesn't break the converter. Will submit this change.

This revision was automatically updated to reflect the committed changes.