[llvm] [AMDGPU] Add option to pre-allocate SGPR spill VGPRs (PR #70626)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 30 07:16:58 PDT 2023


================
@@ -201,6 +205,10 @@ bool SIPreAllocateWWMRegs::runOnMachineFunction(MachineFunction &MF) {
 
   RegClassInfo.runOnMachineFunction(MF);
 
+  bool PreallocateSGPRSpillVGPRs =
+      EnablePreallocateSGPRSpillVGPRs ||
+      MF.getFunction().hasFnAttribute("amdgpu-prealloc-sgpr-spill-vgprs");
----------------
jayfoad wrote:

> Don't see why we would want this controllable by an attribute for migration purposes a flag is sufficient.

For graphics we use the LLVM backed as a shared library so passing options into it  via a command line argument (even if it's just for migration) is a terrible interface.

https://github.com/llvm/llvm-project/pull/70626


More information about the llvm-commits mailing list