[PATCH] D18340: AMDGPU: allow specifying a workgroup size that needs to fit in a compute unit

Bas Nieuwenhuizen via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 22 02:04:04 PDT 2016


bnieuwenhuizen added a comment.

Scratch spilling is per wave though and not per workgroup? Therefore the strides should be the same. For LDS spilling it would be different, but even though we have some functions (SIInstrInfo.cpp, calculateLDSSpillAddress) for LDS spilling I don't see them getting used anywhere?

I will import code + tests from http://reviews.llvm.org/D16834 to handle LDS allocation of variables.


http://reviews.llvm.org/D18340





More information about the llvm-commits mailing list