[PATCH] D28508: [NVPTX] Lower to sqrt.approx and rsqrt.approx under more circumstances.

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

In https://reviews.llvm.org/D28508#644774, @jlebar wrote:

> > The patch only does this transformation with fastmath enabled (or if you pass a special flag to llvm that specifically asks for this transformation):
>
> Hal, maybe this was your point of confusion as well, and if so, I'm sorry about that.  Really bad commit message on my part.


No, I understood that -- I only really care about sqrt and friends when fast-math is enabled. Otherwise the performance is horrible essentially everywhere. ;)


https://reviews.llvm.org/D28508