[PATCH] D63038: [SimplifyLibCalls] powf(x, sitofp(n)) -> powi(x, n)

Sat Jun 8 02:08:10 PDT 2019

xbolva00 added a subscriber: efriedma.
xbolva00 added a comment.

In D63038#1535121 <https://reviews.llvm.org/D63038#1535121>, @nikic wrote:

> Can you please provide some more information under which circumstances `powf(x, (float) y)` will provide a different result than `powi(x, y)`, and which fast-math flags specifically are necessary to make that transform legal?

I don’t have such info, I personally think this is fine also without fast math, but since I cannot say it for sure, In the first version, I put it under fast math.

Maybe some experts like @spatel or @efriedma can confirm that this transformation is always fine?

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

https://reviews.llvm.org/D63038