This is an archive of the discontinued LLVM Phabricator instance.

Interface to get/set profile summary metadata to module
ClosedPublic

Authored by eraman on Mar 4 2016, 1:26 PM.

Details

Summary

This is the first in a series of patches to remove MaxFunctionCount metadata from module and add profile summary metadata instead.

Diff Detail

Repository
rL LLVM

Event Timeline

eraman updated this revision to Diff 49847.Mar 4 2016, 1:26 PM
eraman retitled this revision from to Interface to get/set profile summary metadata to module.
eraman updated this object.
eraman added a reviewer: davidxl.
eraman added a subscriber: llvm-commits.
davidxl edited edge metadata.

Perhaps add a unit test in unittest/ProfileData/InstrProfTest.cpp?

dnovillo edited edge metadata.Mar 10 2016, 10:02 AM

Needs a test. Otherwise, LGTM. Thanks.

eraman updated this revision to Diff 50679.Mar 14 2016, 5:01 PM
eraman edited edge metadata.

Added unit tests.

This revision was automatically updated to reflect the committed changes.