[PATCH] D64099: [InstCombine] pow(C,x) -> exp2(log2(C)*x)

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 3 08:34:11 PDT 2019


evandro added inline comments.


================
Comment at: lib/Transforms/Utils/SimplifyLibCalls.cpp:1355
+    if (Ty->isFloatTy())
+      Log = ConstantFP::get(Ty, std::log(BaseF->convertToFloat()));
+    else if (Ty->isDoubleTy())
----------------
Shouldn't this call and the one below be to `log2()`?


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

https://reviews.llvm.org/D64099





More information about the llvm-commits mailing list