[PATCH] D129381: [AMDGPU][CodeGen] Support (register + immediate) SMRD offsets.

Mon Jul 11 07:39:24 PDT 2022

kosarev added a comment.

In D129381#3639233 <https://reviews.llvm.org/D129381#3639233>, @kosarev wrote:

> I'm looking into what's going on there.

The reason seems to be that with this change in place we generate more spills, which in turn is caused by reduced number of used registers as determined by `AMDGPUResourceUsageAnalysis::analyzeResourceUsage()` -- 102 registers originally vs 96 with the change. I'm not sure how precise the analysis is expected to be and whether this needs a ticket. Would appreciate any advice on this.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129381