[PATCH] D144590: [Clang][OpenMP] Fix shared memory allocation on AMDGPU

2023-03-24 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm added a comment. In D144590#4220648 , @ZwFink wrote: > I wanted to follow up on this. Is anything additional needed from me? This needs a test (I would hope there is already one that needs updating), and also should switch to using PoisonValue in

[PATCH] D144590: [Clang][OpenMP] Fix shared memory allocation on AMDGPU

2023-03-24 Thread Zane Fink via Phabricator via cfe-commits
ZwFink added a comment. Herald added subscribers: jplehr, sunshaoce. I wanted to follow up on this. Is anything additional needed from me? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144590/new/ https://reviews.llvm.org/D144590 _