This is an archive of the discontinued LLVM Phabricator instance.

[BPI] Add multiplication by scalar operators to BranchProbability
ClosedPublic

Authored by skatkov on Apr 20 2017, 7:50 PM.

Details

Summary

This patch just adds two operators to BranchProbability class:
(BP * scalar) and (BP *= scalar).

Diff Detail

Repository
rL LLVM

Event Timeline

skatkov created this revision.Apr 20 2017, 7:50 PM

This patch is actually reviewed as part of https://reviews.llvm.org/D32124

chandlerc accepted this revision.Apr 20 2017, 7:54 PM

Yeah, still LGTM. =D Tiny nit pick though...

unittests/Support/BranchProbabilityTest.cpp
160 ↗(On Diff #96083)

Call it 'Min' to fit with our naming convensions.

This revision is now accepted and ready to land.Apr 20 2017, 7:54 PM
skatkov updated this revision to Diff 96085.Apr 20 2017, 7:57 PM
skatkov marked an inline comment as done.
This revision was automatically updated to reflect the committed changes.