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

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 24 14:00:12 PDT 2023


arsenm added a comment.

In D144590#4220648 <https://reviews.llvm.org/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 instead of UndefValue


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144590



More information about the cfe-commits mailing list