[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