[PATCH] D65979: [InstCombine] Simplify pow() into ldexp()

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 9 09:51:26 PDT 2019


evandro marked an inline comment as done.
evandro added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/SimplifyLibCalls.cpp:1332
+                                    : B.CreateZExt(ExpoI, B.getInt32Ty());
+      return emitBinaryFloatFnCall(ConstantFP::get(Ty, 1.0), ExpoI, TLI,
+                                   LibFunc_ldexp, LibFunc_ldexpf, LibFunc_ldexpl,
----------------
xbolva00 wrote:
> There is one "emit ldexp" old code in SLC which could use emitBinaryFloatFnCall.
Indeed, there are other places that could use `emitBinaryFloatFnCall()`.


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

https://reviews.llvm.org/D65979





More information about the llvm-commits mailing list