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

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 marked an inline comment as done.
xbolva00 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())
----------------
evandro wrote:
> Shouldn't this call and the one below be to `log2()`?
Ah, yes. Fixed.


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

https://reviews.llvm.org/D64099