[PATCH] D17308: AMDGPU: Change how alignment is computed for argument lowering
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 29 12:39:37 PDT 2016
arsenm added inline comments.
================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:565-566
@@ +564,4 @@
+
+ SDValue Load = DAG.getLoad(
+ ISD::UNINDEXED, ExtTy,
+ LoadVT, SL, Chain, Ptr, PtrOffset, PtrInfo, MemVT,
----------------
You should be able to use one of the simpler getLoad operands that doesn't include the ISD::UNINDEXED
================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:1512
@@ -1491,1 +1511,3 @@
+
+ // FIXME: Get alignment from somewhere. Base pointer should be 256 aligned.
SDValue Param = LowerParameter(DAG, MVT::i32, MVT::i32, SL,
----------------
This should be the same MinAlign(Subtarget->getKernargSegmentPtrAlignment(), Offset)
================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:1600-1601
@@ -1577,4 +1599,4 @@
return LowerParameter(DAG, VT, VT, DL, DAG.getEntryNode(),
- SI::KernelInputOffsets::NGROUPS_X, false);
+ SI::KernelInputOffsets::NGROUPS_X, 4, false);
case Intrinsic::r600_read_ngroups_y:
----------------
Should be the same MinAlign again
Repository:
rL LLVM
http://reviews.llvm.org/D17308
More information about the llvm-commits
mailing list