[PATCH] D67199: [InstCombine] Expand the simplification of log()

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Patch uploaded without context.



================
Comment at: llvm/lib/Transforms/Utils/SimplifyLibCalls.cpp:1710
+  if (UnsafeFPShrink && hasFloatVersion(LogNm))
+    Ret = optimizeUnaryDoubleFP(Log, B, true);
 
----------------
Does this do the right thing for non-double libcalls?


================
Comment at: llvm/lib/Transforms/Utils/SimplifyLibCalls.cpp:2888
     case Intrinsic::log:
+    case Intrinsic::log2:
       return optimizeLog(CI, Builder);
----------------
not Intrinsic::log10?


================
Comment at: llvm/test/Transforms/InstCombine/log-pow.ll:93
   %exp = call fast float @exp2f(float %x)
-  %log = call fast float @log10f(float %exp)
   ret float %log
----------------
I'm not sure why you want to remove test coverage for log10f?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67199