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

Justin Lebar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 13 17:38:39 PST 2017


jlebar added a comment.

In https://reviews.llvm.org/D28508#646134, @hfinkel wrote:

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


Heh, well, I expect this to be a strict improvement with fast-math enabled.  :)

I'll check this in when I can.  Thanks for your reviews and attention, everyone.


https://reviews.llvm.org/D28508





More information about the llvm-commits mailing list