[llvm] [AMDGPU] Fix Ins64 clamp in the VOPProfile. NFC. (PR #81925)
    Stanislav Mekhanoshin via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Feb 16 10:31:09 PST 2024
    
    
  
rampitec wrote:
> There's another use of HasIntClamp in VOP3_Pseudo in VOPInstructions.td:
> 
> ```
>   let AsmMatchConverter =
>     !if(isVOP3P,
>         "cvtVOP3P",
>         !if(!or(P.HasModifiers, P.HasOMod, P.HasIntClamp),
>             "cvtVOP3",
>             ""));
> ```
> 
> Does that also need changing?
Yes, nothing breaks if I use HasClamp.
https://github.com/llvm/llvm-project/pull/82020
https://github.com/llvm/llvm-project/pull/81925
    
    
More information about the llvm-commits
mailing list