[PATCH] D78348: [AMDGPU] Add missing AReg classes

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

Missing changes to: AMDGPUAsmPrinter::analyzeResourceUsage(), AMDGPUAsmParser.cpp:getRegClass(), SITargetLowering::getRegForInlineAsmConstraint().



================
Comment at: llvm/lib/Target/AMDGPU/SIRegisterInfo.td:820
+  let CopyCost = 7;
+  let AllocationPriority = 3;
+}
----------------
They all missing let Weight = ... I have somehow missed it.


================
Comment at: llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp:1107
   case AMDGPU::VReg_96RegClassID:
+  case AMDGPU::AReg_96RegClassID:
     return 96;
----------------
Missing AGPR* classes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78348