[PATCH] D82496: [amdgpu] Add codegen support for HIP dynamic shared memory.

[Siu Chi Chan via Phabricator via llvm-commits]

scchan added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:5527
+        return SDValue(
+            DAG.getMachineNode(AMDGPU::GET_GROUPSTATICSIZE, DL, PtrVT), 0);
+      }
----------------
don't you need to make sure whether the static size would give you an offset with the correct alignment?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82496