[PATCH] D47807: Make uitofp and sitofp defined on overflow.

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

Rust currently generates additional code for u128 to f32 casts to work around this problem (see https://github.com/rust-lang/rust/blob/c131bdcaff68d35f96e954baac4340206779335f/src/librustc_codegen_llvm/mir/rvalue.rs#L795). Making this defined would allow directly using uitofp, so this would be a welcome change.


Repository:
  rL LLVM

https://reviews.llvm.org/D47807