[PATCH] D32334: [BPI] Add multiplication by scalar operators to BranchProbability
Chandler Carruth via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 20 19:54:46 PDT 2017
chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.
Yeah, still LGTM. =D Tiny nit pick though...
================
Comment at: unittests/Support/BranchProbabilityTest.cpp:160
+
+ BP MIN(1, 1u << 31);
+
----------------
Call it 'Min' to fit with our naming convensions.
https://reviews.llvm.org/D32334
More information about the llvm-commits
mailing list