[PATCH] D40150: [LibCallSimplifier] fix pow(x, 0.5) -> sqrt() transforms

[Sanjay Patel via Phabricator via llvm-commits]

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