[PATCH] D96643: [AMDGPU] Limit memory scope for scratch, LDS and GDS

Tony Tye via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 13 13:52:37 PST 2021


t-tye added a comment.

In D96643#2561617 <https://reviews.llvm.org/D96643#2561617>, @rampitec wrote:

> LGTM, thanks Tony!
> It is up to you to use min() or if(). I will postpone my patch to rebase on top your change as I expect to have zero test changes with my patch after this one and want to make sure this expectation is correct.

I made the changes, thanks. I am waiting to get executable test results but Gerrit is currently down.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96643



More information about the llvm-commits mailing list