[PATCH] D85271: AMDGPU/GlobalISel: Make s16 phi legal

[Jay Foad via Phabricator via llvm-commits]

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

I don't know if I understand all the impliciations, but the diffs look fine to me.



================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:446
   getActionDefinitionsBuilder(G_PHI)
-    .legalFor({S32, S64, V2S16, V4S16, S1, S128, S256})
+    .legalFor({S32, S64, V2S16, S16, V4S16, S1, S128, S256})
     .legalFor(AllS32Vectors)
----------------
It doesn't really matter but this seems like an odd place to insert "S16" into the list.


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

https://reviews.llvm.org/D85271