git: 595a5d5fa099 - main - net/openmpi: Fix shortfloat support
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
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 "