[PATCH] D11442: Create a utility function normalizeEdgeWeights() in BranchProbabilityInfo that is used to normalize a list of weights so that the sum of them does not exceed UINT32_MAX.

David davidxl at google.com
Thu Jul 23 12:30:32 PDT 2015


davidxl added inline comments.

================
Comment at: include/llvm/Analysis/BranchProbabilityInfo.h:165
@@ +164,3 @@
+  // If the computed sum fits in 32-bits, we're done.
+  if (Sum <= UINT32_MAX)
+    return 1;
----------------
Why not doing normalization unconditionally to avoid precision loss introduced by incremental update (for small weights)?


http://reviews.llvm.org/D11442







More information about the llvm-commits mailing list