git: 595a5d5fa099 - main - net/openmpi: Fix shortfloat support

From: Vladimir Druzenko <vvd_at_FreeBSD.org>
Date: Sun, 18 Aug 2024 11:50:34 UTC
The branch main has been updated by vvd:

URL: https://cgit.FreeBSD.org/ports/commit/?id=595a5d5fa099770ebece23deb61fa364eba955db

commit 595a5d5fa099770ebece23deb61fa364eba955db
Author:     Laurent Chardon <laurent.chardon@gmail.com>
AuthorDate: 2024-08-18 11:42:14 +0000
Commit:     Vladimir Druzenko <vvd@FreeBSD.org>
CommitDate: 2024-08-18 11:42:14 +0000

    net/openmpi: Fix shortfloat support
    
    - Improve the test for half precision support. The old test gave
    incorrect results, in particular for older compilers.
    The test is now compiler independent since both gcc and clang
    have supported __Float16 for a long while.
    - Remove unused code for plist substitution of NO_MCA_PATCHER_OVERWRITE
    
    PR:     280879
---
 net/openmpi/Makefile | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/net/openmpi/Makefile b/net/openmpi/Makefile
index eef31aca20a2..e41a44ccb700 100644
--- a/net/openmpi/Makefile
+++ b/net/openmpi/Makefile
@@ -1,5 +1,6 @@
 PORTNAME=	openmpi
-PORTVERSION=	5.0.5
+DISTVERSION=	5.0.5
+PORTREVISION=	1
 CATEGORIES=	net parallel
 MASTER_SITES=	https://download.open-mpi.org/release/open-mpi/v${PORTVERSION:R}/
 
@@ -88,13 +89,9 @@ SLURM_CONFIGURE_WITH=	slurm
 FCFLAGS+=	-fallow-argument-mismatch
 .endif
 
-.if ${ARCH} != aarch64 && ${ARCH} != amd64 && ${ARCH} != i386 && !${ARCH:Mpowerpc*}
-PLIST_SUB+=	NO_MCA_PATCHER_OVERWRITE="@comment "
-.else
-PLIST_SUB+=	NO_MCA_PATCHER_OVERWRITE=""
-.endif
-
-.if ${COMPILER_TYPE} == clang && ${COMPILER_VERSION} >= 170 && !${ARCH:Mpowerpc*}
+# Both clang and gcc support half precision on these platforms
+# Note: RISC-V can support half precision with Zhf extension
+.if ${ARCH:Maarch64} || ${ARCH:Mamd64}
 PLIST_SUB+=	SHORTFLOAT=""
 .else
 PLIST_SUB+=	SHORTFLOAT="@comment "