[PATCH] D40150: [LibCallSimplifier] fix pow(x, 0.5) -> sqrt() transforms
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 17 06:47:08 PST 2017
spatel added inline comments.
================
Comment at: lib/Transforms/Utils/SimplifyLibCalls.cpp:1080
+ // TODO: There is some subset of 'fast' under which these transforms should
+ // be allowed.
+ if (!Pow->isFast())
----------------
efriedma wrote:
> Probably something like ninf nsz? But let's do that in a followup.
That might be it. As you can tell from the other tests that I added, I initially set out to make this transform a potential test of the meaning of the new 'afn' FMF...but then I got stuck in another batch of errno quicksand. :)
https://reviews.llvm.org/D40150
More information about the llvm-commits
mailing list