[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