[llvm] AMDGPU: Use less surprising form of ConstantFP::get (PR #139248)
via llvm-commits
llvm-commits at lists.llvm.org
Fri May 9 05:02:00 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-amdgpu
Author: Matt Arsenault (arsenm)
<details>
<summary>Changes</summary>
Avoid deducing the IR type from the APFloat type
---
Full diff: https://github.com/llvm/llvm-project/pull/139248.diff
1 Files Affected:
- (modified) llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp (+2-2)
``````````diff
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp b/llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp
index 8679a7cd0f88d..e76396f6ffbb0 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp
@@ -906,8 +906,8 @@ GCNTTIImpl::instCombineIntrinsic(InstCombiner &IC, IntrinsicInst &II) const {
if (const ConstantFP *C2 = dyn_cast<ConstantFP>(Src2)) {
APFloat Result = fmed3AMDGCN(C0->getValueAPF(), C1->getValueAPF(),
C2->getValueAPF());
- return IC.replaceInstUsesWith(
- II, ConstantFP::get(IC.Builder.getContext(), Result));
+ return IC.replaceInstUsesWith(II,
+ ConstantFP::get(II.getType(), Result));
}
}
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/139248
More information about the llvm-commits
mailing list