[PATCH] D54749: Saturating float to int casts: Basics [1/n]

[Nikita Popov via Phabricator via llvm-commits]

nikic marked 2 inline comments as done.
nikic added inline comments.


================
Comment at: docs/LangRef.rst:13105
+
+      declare i32 @llvm.fptoui.sat.f32.i32(float %f)
+      declare i19 @llvm.fptoui.sat.f64.i19(double %f)
----------------
t.p.northover wrote:
> What's going on here? All the other intrinsics I know specify the return type first in the prototype. I didn't even think it was something you could override.
You're absolutely right, I've flipped the order now, here and in tests. It looks like LLVM is very forgiving of incorrect intrinsic name suffixes and just normalizes them to be correct.


https://reviews.llvm.org/D54749