[PATCH] D115402: AMDGPU: Add second emergency slot for SGPR to vmem for large frames

Carl Ritson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 9 01:20:29 PST 2021


critson added a comment.

This seems fine to me, but I am slightly confused the description mentions Wave32, but the test is for GFX9 Wave64.
I can clearly see the test exercises the new code, but do we need to also test/implement the Wave32 part?


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

https://reviews.llvm.org/D115402



More information about the llvm-commits mailing list