forked from OSchip/llvm-project
R600/SI: Wrap local memory pointer in AssertZExt on SI
These pointers are really just offsets and they will always be less than 16-bits. Using AssertZExt allows us to use computeKnownBits to prove that these values are positive. We will use this information in a later commit. llvm-svn: 216277
This commit is contained in:
parent
0510514e36
commit
ca7ecf3dfa
|
@ -504,6 +504,18 @@ SDValue SITargetLowering::LowerFormalArguments(
|
|||
SDValue Arg = LowerParameter(DAG, VT, MemVT, DL, DAG.getRoot(),
|
||||
36 + VA.getLocMemOffset(),
|
||||
Ins[i].Flags.isSExt());
|
||||
|
||||
const PointerType *ParamTy =
|
||||
dyn_cast<PointerType>(FType->getParamType(Ins[i].OrigArgIndex));
|
||||
if (Subtarget->getGeneration() == AMDGPUSubtarget::SOUTHERN_ISLANDS &&
|
||||
ParamTy && ParamTy->getAddressSpace() == AMDGPUAS::LOCAL_ADDRESS) {
|
||||
// On SI local pointers are just offsets into LDS, so they are always
|
||||
// less than 16-bits. On CI and newer they could potentially be
|
||||
// real pointers, so we can't guarantee their size.
|
||||
Arg = DAG.getNode(ISD::AssertZext, DL, Arg.getValueType(), Arg,
|
||||
DAG.getValueType(MVT::i16));
|
||||
}
|
||||
|
||||
InVals.push_back(Arg);
|
||||
continue;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue