[PATCH] D62430: [PPC] Correctly adjust branch probability in PPCReduceCRLogicals

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 24 15:29:12 PDT 2019


hfinkel added inline comments.


================
Comment at: include/llvm/Support/BranchProbability.h:161
+  BranchProbability sqrt() const {
+    double P = (double)N / (double)D;
+    P = std::sqrt(P);
----------------
This would be the first use of floating-point here, right? Can we do this any other way?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62430/new/

https://reviews.llvm.org/D62430





More information about the llvm-commits mailing list