[PATCH] D63782: [FPEnv] Add fptosi and fptoui constrained intrinsics

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

The problem with poison is that it eventually leads to UB, and then your program has no defined meaning.  Practically, it might mean some codepath that involves a call to llvm.experimental.constrained.fptosi.i32.f64 could get folded away because it's provably UB, or something like that.

In any case, we should explicitly state what happens, since it isn't specified by IEEE754.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63782