[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