[PATCH] D49273: [InstCombine] Expand the simplification of pow() into exp2()

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 7 13:37:20 PDT 2019


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

In D49273#1619685 <https://reviews.llvm.org/D49273#1619685>, @xbolva00 wrote:

> Hello, ldexp seems to be faster here. Did you try it?
>
> pow(2, e) to ldexp(1, e)


`ldexp()` restricts the exponent to integers, so only then is it potentially faster.

Will think about it.

Thank you.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D49273





More information about the llvm-commits mailing list