[PATCH] D15548: Remove the restriction that known and unknown probabilities cannot coexist when being normalized.
Cong Hou via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 17 14:12:44 PST 2015
congh marked an inline comment as done.
================
Comment at: include/llvm/Support/BranchProbability.h:195
@@ +194,3 @@
+
+ if (UnknownProbCount > 0 && Sum > 0) {
+ BranchProbability UnknownProb = BranchProbability::getZero();
----------------
davidxl wrote:
> why guarded with Sum > 0?
After reviewing the patch, I think this guard should be removed. Thanks for pointing it out!
http://reviews.llvm.org/D15548
More information about the llvm-commits
mailing list