This is an archive of the discontinued LLVM Phabricator instance.

CodeGen: BlockPlacement: Don't always tail-duplicate with no other successor.
ClosedPublic

Authored by iteratee on Mar 31 2017, 5:41 PM.

Details

Reviewers
davidxl
Summary

The math works out where it can actually be counter-productive. The probability
calculations correctly handle the case where the alternative is 0 probability,
rely on those calculations.

Diff Detail

Event Timeline

iteratee created this revision.Mar 31 2017, 5:41 PM
davidxl accepted this revision.Apr 1 2017, 12:20 PM

lgtm

This revision is now accepted and ready to land.Apr 1 2017, 12:20 PM
iteratee updated this revision to Diff 94125.Apr 4 2017, 2:37 PM
iteratee closed this revision.May 2 2017, 4:34 PM

Committed as rL299892