[llvm] [AMDGPU] Fix an issue that wrong index is used in calculation of byte provider when the op is extract_vector_elt (PR #91697)
Jeffrey Byrnes via llvm-commits
llvm-commits at lists.llvm.org
Mon May 20 15:11:15 PDT 2024
jrbyrnes wrote:
> Why do we want to differentiate them here?
We don't gain anything from looking into the vectors for scalarsize >= 32. calculateSrcByte will just return the Op since vector handling hasn't been built yet, and then we will need to re-extract the dword when building the perm https://github.com/llvm/llvm-project/blob/main/llvm/lib/Target/AMDGPU/SIISelLowering.cpp#L12193 -- by disallowing them, we avoid some unecessary computation. Additionally, it does not introduce the regression mentioned above.
https://github.com/llvm/llvm-project/pull/91697
More information about the llvm-commits
mailing list