[PATCH] D123941: AMDGPU: Fix allocating GDS globals to LDS offsets

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec accepted this revision.
rampitec added a comment.
This revision is now accepted and ready to land.

LGTM with the assert added.



================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp:71
+  } else {
+    Offset = StaticGDSSize = alignTo(StaticGDSSize, Alignment);
+    StaticGDSSize += DL.getTypeAllocSize(GV.getValueType());
----------------
Assert the AS is REGION?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123941/new/

https://reviews.llvm.org/D123941