llvm dev - May 2016 - AMDGPUPromoteAlloca assume 3-dims enabled?

hi, list,

I found AMDGPUPromoteAlloca calculates newly ptr as follows:

  std::tie(TCntY, TCntZ) = getLocalSizeYZ(Builder);

  Value *TIdX = getWorkitemID(Builder, 0);

  Value *TIdY = getWorkitemID(Builder, 1);

  Value *TIdZ = getWorkitemID(Builder, 2);


  Value *Tmp0 = Builder.CreateMul(TCntY, TCntZ, "", true, true);

  Tmp0 = Builder.CreateMul(Tmp0, TIdX);

  Value *Tmp1 = Builder.CreateMul(TIdY, TCntZ, "", true, true);

  Value *TID = Builder.CreateAdd(Tmp0, Tmp1);

  TID = Builder.CreateAdd(TID, TIdZ);


it assumes that we enable 3 dims already?


actually, it's not the case for SI. SI only enable dim-x

for non-shader programs by default(SIMachinefunctionInfo.cpp) . does
it conflict?


thanks,

--lx
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20160531/b28fc0b4/attachment.html>