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

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 4 16:54:21 PDT 2019


evandro marked 6 inline comments as done.
evandro added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/SimplifyLibCalls.cpp:2888
     case Intrinsic::log:
+    case Intrinsic::log2:
       return optimizeLog(CI, Builder);
----------------
efriedma wrote:
> evandro wrote:
> > efriedma wrote:
> > > not Intrinsic::log10?
> > We don't have one yet.
> LangRef says we have one; is it wrong?
No, but perhaps I am.


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

https://reviews.llvm.org/D67199





More information about the llvm-commits mailing list