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] Rename AMDGPULoadTr intrinsic class. NFC. #79394

Merged
merged 2 commits into from
Jan 25, 2024
Merged

Conversation

changpeng
Copy link
Contributor

This class is not specific to "Tr"(Transpose). It is
just for a normal load.

  This class is not specific to "Tr"(Transpose). It is
just for a normal load.
@llvmbot
Copy link
Collaborator

llvmbot commented Jan 25, 2024

@llvm/pr-subscribers-llvm-ir

@llvm/pr-subscribers-backend-amdgpu

Author: Changpeng Fang (changpeng)

Changes

This class is not specific to "Tr"(Transpose). It is
just for a normal load.


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

1 Files Affected:

  • (modified) llvm/include/llvm/IR/IntrinsicsAMDGPU.td (+2-2)
diff --git a/llvm/include/llvm/IR/IntrinsicsAMDGPU.td b/llvm/include/llvm/IR/IntrinsicsAMDGPU.td
index 9eb1ac8e27befb1..1fbaf569c321335 100644
--- a/llvm/include/llvm/IR/IntrinsicsAMDGPU.td
+++ b/llvm/include/llvm/IR/IntrinsicsAMDGPU.td
@@ -2755,7 +2755,7 @@ def int_amdgcn_global_atomic_fmax_num : AMDGPUAtomicRtn<llvm_anyfloat_ty>;
 
 def int_amdgcn_atomic_cond_sub_u32 : AMDGPUAtomicRtn<llvm_i32_ty>;
 
-class AMDGPULoadTr<LLVMType ptr_ty>:
+class AMDGPULoadIntrinsic<LLVMType ptr_ty>:
   Intrinsic<
     [llvm_any_ty],
     [ptr_ty],
@@ -2775,7 +2775,7 @@ class AMDGPULoadTr<LLVMType ptr_ty>:
 // <4 x half>   @llvm.amdgcn.global.load.tr.v4f16(ptr addrspace(1))  -> global_load_tr_b128
 // <4 x bfloat> @llvm.amdgcn.global.load.tr.v4bf16(ptr addrspace(1)) -> global_load_tr_b128
 
-def int_amdgcn_global_load_tr : AMDGPULoadTr<global_ptr_ty>;
+def int_amdgcn_global_load_tr : AMDGPULoadIntrinsic<global_ptr_ty>;
 
 //===----------------------------------------------------------------------===//
 // Deep learning intrinsics.

@changpeng changpeng merged commit aa64a82 into llvm:main Jan 25, 2024
3 of 4 checks passed
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