[PATCH] D40148: [AMDGPU] Fix SITargetLowering::lowerEXTRACT_VECTOR_ELT for constant type

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 16 18:10:59 PST 2017


arsenm requested changes to this revision.
arsenm added a comment.
This revision now requires changes to proceed.

SplitInteger,
where default pointer type is used for the constant 16.

This is wrong. This should not be using getIntPtrConstant. For AMDGPU the shift type is always 32. If somewhere is creating a shift mask with something other than getScalarShiftAmountTy, that is a bug there.


https://reviews.llvm.org/D40148





More information about the llvm-commits mailing list