[llvm] [AMDGPU] Combine VGPRSrc and VGPROp definitions into VGPROp (PR #157516)

Stanislav Mekhanoshin via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 8 10:56:23 PDT 2025


================
@@ -2681,7 +2681,7 @@ class VOPProfile <list<ValueType> _ArgVT, bit _EnableClamp = 0> {
   field RegisterOperand Src0DPP = getVregSrcForVT<Src0VT>.ret;
   field RegisterOperand Src1DPP = getVregSrcForVT<Src1VT>.ret;
   field RegisterOperand Src2DPP = getVregSrcForVT<Src2VT>.ret;
-  field RegisterOperand Src0VOP3DPP = VGPRSrc_32;
+  field RegisterOperand Src0VOP3DPP = VGPROp_32;
----------------
rampitec wrote:

It is different from not yet upstreamed downstream code:
```
field RegisterOperand Src0VOP3DPP = getVGPRSrcForVT<Src0VT>.ret;
```
Maybe you can pull that helper from donwstream while you are here? We will need to do something about it on merge otherwise.

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


More information about the llvm-commits mailing list