[PATCH] D83880: AMDGPU/GlobalISel: Legalize s64->s16 G_SITOFP/G_UITOFP

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 15 08:26:14 PDT 2020


foad added a comment.

> Legailize as s64->s32 G_SITOFP/G_UITOFP followed by s32->s16 G_FPTRUNC.

It seems like this would give the wrong result in some cases because of the double rounding. What does SelectionDAG do?


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

https://reviews.llvm.org/D83880





More information about the llvm-commits mailing list