[PATCH] D81576: AMDGPU/GlobalISel: Fix some legalization of < dword vector stores

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 15 07:33:28 PDT 2020


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:125
       // <2 x s8> -> s16
-      assert(Size == 16);
-      CoercedTy = LLT::scalar(16);
+      // <4 x s8> -> s32
+      CoercedTy = LLT::scalar(Size);
----------------
This comment doesn't make sense given the `Size < 32` condition two lines above.


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

https://reviews.llvm.org/D81576





More information about the llvm-commits mailing list