[llvm] [AMDGPU] Use correct vector elt type when shrinking mfma scale (PR #123043)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 15 03:17:59 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-amdgpu
Author: Fraser Cormack (frasercrmck)
<details>
<summary>Changes</summary>
This might be a copy/paste error. I don't think this an issue in practice as the builtins/intrinsics are only legal with identical vector element types.
---
Full diff: https://github.com/llvm/llvm-project/pull/123043.diff
1 Files Affected:
- (modified) llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp (+1-1)
``````````diff
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp b/llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp
index 8b1b398606583e..bac3bb5fde7b02 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp
@@ -1346,7 +1346,7 @@ GCNTTIImpl::instCombineIntrinsic(InstCombiner &IC, IntrinsicInst &II) const {
if (Src1Ty->getNumElements() > Src1NumElts) {
Src1 = IC.Builder.CreateExtractVector(
- FixedVectorType::get(Src0Ty->getElementType(), Src1NumElts), Src1,
+ FixedVectorType::get(Src1Ty->getElementType(), Src1NumElts), Src1,
IC.Builder.getInt64(0));
MadeChange = true;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/123043
More information about the llvm-commits
mailing list