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

Kevin P. Neal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 26 09:50:30 PDT 2019


kpn added a comment.

In D63782#1558430 <https://reviews.llvm.org/D63782#1558430>, @efriedma wrote:

> What happens if the input float is out of range?  fptosi/fptoui instructions produce poison; not sure if you want that here.


I'm not well versed in poison. Are you worried about constant folding away the fptoXi? If a run-time conversion could raise a trap I expect constant folding to be skipped. Would that avoid poison?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63782





More information about the llvm-commits mailing list