[LibCallSimplifier] use instruction-level fast-math-flags to transform sqrt calls
authorSanjay Patel <spatel@rotateright.com>
Mon, 11 Jan 2016 22:34:19 +0000 (22:34 +0000)
committerSanjay Patel <spatel@rotateright.com>
Mon, 11 Jan 2016 22:34:19 +0000 (22:34 +0000)
commit2234dddb07dfaf9ff292ee96efb28bc82a91b66f
tree511c00685e181d2d99515d1994c8ba64d0c57e16
parentd24b395f8b297e6437c00d4fbafeac2e69cdf9b7
[LibCallSimplifier] use instruction-level fast-math-flags to transform sqrt calls

This is a continuation of adding FMF to call instructions:
http://reviews.llvm.org/rL255555

The intent of the patch is to preserve the current behavior of the transform except
that we use the sqrt instruction's 'fast' attribute as a trigger rather than the
function-level attribute.

But this raises a bug noted by the new FIXME comment.

In order to do this transform:
sqrt((x * x) * y) ---> fabs(x) * sqrt(y)

...we need all of the sqrt, the first fmul, and the second fmul to be 'fast'.
If any of those ops is strict, we should bail out.

Differential Revision: http://reviews.llvm.org/D15937

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@257400 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Transforms/Utils/SimplifyLibCalls.cpp
test/Transforms/InstCombine/fast-math.ll
test/Transforms/InstCombine/inline-intrinsic-assert.ll
test/Transforms/InstCombine/no_cgscc_assert.ll