[PATCH] D127052: [amdgpu][nfc] Allocate kernel-specific LDS struct deterministically

Jon Chesterfield via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 6 12:13:21 PDT 2022


JonChesterfield added a comment.

I think the layout this chooses is always one of the layouts that could have been picked by the iteration order before. Dynamic LDS alignment gets set when encountering the dynamic variable which could be after the other two were allocated.

I'll write some IR to asm cases that would previously fail non-deterministically but will now pass reliably.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127052



More information about the llvm-commits mailing list