diff options
author | David Callahan <dcallahan@fb.com> | 2016-08-05 18:38:19 +0000 |
---|---|---|
committer | David Callahan <dcallahan@fb.com> | 2016-08-05 18:38:19 +0000 |
commit | c1c810de0b55a6d4786974a295d7ef665a860f93 (patch) | |
tree | 04cd7a833b18744bf0c4e259f9c2fee28d43be5c /llvm/docs/tutorial | |
parent | b0b4e86215fe2815ffed9e70c810b08c0ca4e47f (diff) | |
download | bcm5719-llvm-c1c810de0b55a6d4786974a295d7ef665a860f93.tar.gz bcm5719-llvm-c1c810de0b55a6d4786974a295d7ef665a860f93.zip |
[AutoFDO] Fix handling of empty profiles
Summary:
If a profile has no samples for a function, then the function "entry count" is set to the value 0. Several places in the code test that if the Function::getEntryCount is defined at all. Here we change to treat a 0 entry count the same as undefined.
In particular, this fixes a problem in getLayoutSuccessorProbThreshold in MachineBlockPlacement.cpp where we use a different and inferior heuristic for laying out basic blocks.
Reviewers: danielcdh, dnovillo
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D23082
llvm-svn: 277849
Diffstat (limited to 'llvm/docs/tutorial')
0 files changed, 0 insertions, 0 deletions