Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AMDGPU] Use HasClamp instead of HasIntClamp in VOP3_Pseudo. NFC. #82020

Merged
merged 1 commit into from
Feb 17, 2024

Conversation

rampitec
Copy link
Collaborator

There is no real reason to differentiate.

@llvmbot
Copy link
Collaborator

llvmbot commented Feb 16, 2024

@llvm/pr-subscribers-backend-amdgpu

Author: Stanislav Mekhanoshin (rampitec)

Changes

There is no real reason to differentiate.


Full diff: https://github.com/llvm/llvm-project/pull/82020.diff

1 Files Affected:

  • (modified) llvm/lib/Target/AMDGPU/VOPInstructions.td (+1-1)
diff --git a/llvm/lib/Target/AMDGPU/VOPInstructions.td b/llvm/lib/Target/AMDGPU/VOPInstructions.td
index f2bb58ed4c3b56..dc6a48b1df7472 100644
--- a/llvm/lib/Target/AMDGPU/VOPInstructions.td
+++ b/llvm/lib/Target/AMDGPU/VOPInstructions.td
@@ -161,7 +161,7 @@ class VOP3_Pseudo <string opName, VOPProfile P, list<dag> pattern = [],
   let AsmMatchConverter =
     !if(isVOP3P,
         "cvtVOP3P",
-        !if(!or(P.HasModifiers, P.HasOMod, P.HasIntClamp),
+        !if(!or(P.HasModifiers, P.HasOMod, P.HasClamp),
             "cvtVOP3",
             ""));
 }

@rampitec rampitec merged commit f847c72 into llvm:main Feb 17, 2024
6 checks passed
@rampitec rampitec deleted the intclamp-asm-converter branch February 17, 2024 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants