[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