Hi!

This issue may be a good introductory issue for people new to working on LLVM. If you would like to work on this issue, your first steps are:

1. Assign the issue to you.
2. Fix the issue locally.
3. Run the test suite locally.
   3.1) Remember that the subdirectories under test/ create fine-grained testing targets, so you can
   e.g. use make check-clang-ast to only run Clang's AST tests.
4. Create a git commit
5. Run git clang-format HEAD~1 to format your changes.
6. Submit the patch to Phabricator.
   6.1) Detailed instructions can be found here

For more instructions on how to submit a patch to LLVM, see our documentation.

If you have any further questions about this issue, don't hesitate to ask via a comment on this Github issue.

@llvm/issue-subscribers-good-first-issue

Candidate Patch: https://reviews.llvm.org/D150372

Candidate Patch: https://reviews.llvm.org/D150372

@e-kud Do you intend to resurrect this patch or might somebody else continue it?

@RKSimon let me prioritize this issue. I remember there were two problems: 1) not all fptosi+min+max can be combined into fptosi_sat but they should be 2) one of fp-int type combination wasn't convenient to support. I've tried to fix them both at once, that's why it has been stuck. I'll concentrate on fptosi_sat exclusively.