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

[Ulrich Weigand via Phabricator via llvm-commits]

uweigand added a comment.

I've now posted a patch implementing something along those lines here: https://reviews.llvm.org/D65226

The only difference is that if the strict operation is to fall back to the non-strict operation, and that non-strict operation is marked Custom, I'm now **not** attempting to call that Custom handler.  This is mostly because I haven't found any case where this would actually be invoked today (and therefore couldn't test any code added to handle this case), and also because it doesn't really make much sense.  Going forward, when this will show up (e.g. because of fptosi/fptoui), the target simply should mark the strict operation as Custom as well.  (This will actually work correctly with D65226 <https://reviews.llvm.org/D65226>.)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63782