[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