[PATCH] D60021: InstSimplify: Fold round intrinsics from sitofp

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

I'm still a bit unsure about this. I could be simply confused though.
I agree that it is fine for those integral types when `iN.SINT_MAX() <= float.MAX() && iN.SINT_MIN() >= float.MIN()`.
That means, we are good for up to i128 for `float` / i1024 for `double` or so.
But what about larger types? I don't think `[su]itofp` clamps?


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

https://reviews.llvm.org/D60021