[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