[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