This is an archive of the discontinued LLVM Phabricator instance.

[PGO] make the number of records for each value site metada adjustable
ClosedPublic

Authored by xur on Feb 10 2016, 11:11 AM.

Details

Summary

Add a parameter in annotateValueSite() to control the max number of records written
to the value profile meta data for each value site. The default is kept as the
current value of 3.

Diff Detail

Event Timeline

xur updated this revision to Diff 47489.Feb 10 2016, 11:11 AM
xur retitled this revision from to [PGO] make the number of records for each value site metada adjustable.
xur updated this object.
xur added reviewers: davidxl, betulb.
xur added a subscriber: llvm-commits.
davidxl edited edge metadata.Feb 10 2016, 11:14 AM

need to add a unit test.

xur updated this revision to Diff 47497.Feb 10 2016, 11:40 AM
xur edited edge metadata.

I expanded an existing unit test to test the added feature. I hope this is ok.

davidxl accepted this revision.Feb 10 2016, 12:14 PM
davidxl edited edge metadata.

lgtm

This revision is now accepted and ready to land.Feb 10 2016, 12:14 PM
This revision was automatically updated to reflect the committed changes.