[PATCH] D87877: [InstCombine] Fix errno bug in pow expansion to sqrt

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 22 18:20:16 PDT 2020


hubert.reinterpretcast added a comment.

In D87877#2288974 <https://reviews.llvm.org/D87877#2288974>, @efriedma wrote:

> I haven't been following this closely, but is there some reason we can't transform `powf(x, 0.5)` to `sqrt(x == -infinity ? qnan : x)`?

Hmm, moving the `select` inwards should work, yes: `sqrt(x == -infinity ? +infinity : x)`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87877/new/

https://reviews.llvm.org/D87877



More information about the llvm-commits mailing list