[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