[PATCH] D130784: [AMDGPU] Support LDS spilling

[Venkata Ramanaiah Nalamothu via Phabricator via llvm-commits]

RamNalamothu added a comment.

@scott.linder

I don't think there are any issues.

Looks like it's possible to calculate LDS offset of the spill and buildCFIForVGPRToVMEMSpill() needs to be specialized to include DW_OP_LLVM_form_aspace_address for LDS.



================
Comment at: llvm/lib/Target/AMDGPU/SIRegisterInfo.cpp:1290
+
+    // FIXME: If we could prove that the are no m0 defs/uses between two LDS
+    // spill instructions we could avoid doing some save/restore.
----------------
s/the are no/there are no


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130784