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

Fri Jun 21 12:43:18 PDT 2019

spatel added a comment.

In D63038#1554099 <https://reviews.llvm.org/D63038#1554099>, @xbolva00 wrote:

> In D63038#1554089 <https://reviews.llvm.org/D63038#1554089>, @efriedma wrote:
>
> > It would be nice to use the exact necessary fast-math flags here, while we're thinking about it, instead of just "isFast()".  From the discussion, it seems like we only need "afn"?
>
>
> Okey. Maybe @spatel could help us with fast flags, whether afn is enough.

Yes, I think 'afn' gives us the freedom for this sort of thing. As a practical matter, I'm not sure if clang has the means to turn on 'afn' without the entirety of "-ffast-math", but that may change in the future.

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

https://reviews.llvm.org/D63038