[PATCH] D102353: [AArch64] Lower fpto*i.sat intrinsics.
Jacob Bramley via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 13 01:10:30 PDT 2021
jbramley added a comment.
In D102353#2755176 <https://reviews.llvm.org/D102353#2755176>, @nikic wrote:
> For the record, there is also the somewhat stale D86078 <https://reviews.llvm.org/D86078> for this.
Thanks, I wasn't aware of that. At a glance, it looks like that patch implements `DstWidth != SatWidth` and vector types; I left that as a TODO, hoping to seek feedback on the overall approach first. I think they'll be fairly clean incremental changes but I can roll them into this patch if that's preferred.
By the way, the use-case for this is Rust, which has well-defined sematics for FP-to-integer casts so this becomes a fairly common operation.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102353/new/
https://reviews.llvm.org/D102353
More information about the llvm-commits
mailing list