[PATCH] D84010: AMDGPU/GlobalISel: Legalize s16->s64 G_FPTOSI/G_FPTOUI

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 17 10:24:44 PDT 2020


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

LGTM with nit



================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:658
+    .customFor({{S64, S64}})
+    .narrowScalarFor({{S64, S16}},changeTo(0, S32));
   if (ST.has16BitInsts())
----------------
Missing space after ,


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

https://reviews.llvm.org/D84010





More information about the llvm-commits mailing list