[PATCH] D27028: Add intrinsics for constrained floating point operations

[Andy Kaylor via Phabricator via llvm-commits]

andrew.w.kaylor added a comment.

In https://reviews.llvm.org/D27028#634214, @b-sumner wrote:

> FWIW, rounding controls are needed for llvm.fma.*, llvm.fmuladd.*, and llvm.sqrt.*


I agree these are needed.  I've added a comment in the .td file and intend to add variations of those intrinsics in a later revision.

In https://reviews.llvm.org/D27028#634214, @b-sumner wrote:

> Also, I don't see any reason to cover frem, but I also don't understand how frem ever got to be an instruction instead of a standard C library intrinsic.


I'm not sure it actually happens, but I would think that frem is potentially subject to the same sorts of constant folding that can be done with fdiv.


Repository:
  rL LLVM

https://reviews.llvm.org/D27028