[PATCH] D54749: Saturating float to int casts.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/docs/LangRef.rst:16197
+  result type (this includes positive infinity), the largest representable
+  integer is returned.
+- Otherwise, the result of rounding the argument towards zero is returned.
----------------
representable unsigned integer


================
Comment at: llvm/docs/LangRef.rst:16247
+  result type (this includes negative infinity), the smallest representable
+  integer is returned.
+- If the argument is larger than the largest representable integer of the
----------------
representable signed integer


================
Comment at: llvm/docs/LangRef.rst:16250
+  result type (this includes positive infinity), the largest representable
+  integer is returned.
+- Otherwise, the result of rounding the argument towards zero is returned.
----------------
representable signed integer


================
Comment at: llvm/include/llvm/CodeGen/ISDOpcodes.h:737
+  /// The width in operand 1 may be smaller than the scalar result type width
+  /// as a result of integer type legalization.
+  FP_TO_SINT_SAT,
----------------
Maybe explicitly say that the op1 width may be smaller or equal (but not larger) to the result width


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D54749