[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:55:24 PDT 2019


evandro added a comment.

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

> We have exp to ldexp fold, so we indirectly fold pow integer case to ldexp. I think it is okay as is.


The test `pow_fp_int.ll` indicates that it doesn't fold `exp2()` into `ldexp()` then.


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