[PATCH] D57359: [GlobalISel] Introduce a G_FSQRT generic instruction

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 30 10:04:17 PST 2019


arsenm added a comment.

In D57359#1377345 <https://reviews.llvm.org/D57359#1377345>, @paquette wrote:

> So, is the consensus here that we
>
> - Don't split this into two opcodes (for now)
> - Document the behaviour of G_FSQRT wrt negative values and errno ?


Sounds right to me. Since the intrinsic definition was fixed, I don't see a reason to ever have two opcodes.


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

https://reviews.llvm.org/D57359





More information about the llvm-commits mailing list