[llvm] [AMDGPU][NFCI] Add IEEEMinimumMaximumInsts SubtargetFeature (PR #141081)
Mirko BrkuĊĦanin via llvm-commits
llvm-commits at lists.llvm.org
Mon May 26 09:44:26 PDT 2025
================
@@ -2098,7 +2098,7 @@ AMDGPULegalizerInfo::AMDGPULegalizerInfo(const GCNSubtarget &ST_,
G_SADDO, G_SSUBO})
.lower();
- if (ST.hasIEEEMinMax()) {
+ if (ST.hasIEEEMinMaxInsts()) {
----------------
mbrkusanin wrote:
If you mean v_pk_minimum3_f16 and v_pk_maximum3_f16, then FeatureMinimum3Maximum3PKF16 already exists.
New IEEEMinimumMaximumInsts will include v_pk_minimum_f16 and v_pk_maximum_f16.
You want to have just v2f16 legal when only FeatureMinimum3Maximum3PKF16 exists because of MinimumMaximumByMinimum3Maximum3VOP3P? That would not be NFC so I do not want to put it here.
https://github.com/llvm/llvm-project/pull/141081
More information about the llvm-commits
mailing list