From: Jeremie Courreges-Anglas <jca@wxcvbn.org>
Subject: Re: UPDATE: highway - riscv64
To: Brad Smith <brad@comstyle.com>
Cc: ports@openbsd.org
Date: Thu, 8 Oct 2026 00:40:13 +0200

On Fri, Sep 25, 2026 at 08:39:11PM -0400, Brad Smith wrote:
> Now that Clang has been fixed this now builds fine on riscv64.

At the risk of stating the obvious: I added these lines, please Cc
when removing them.

> Index: devel/highway/Makefile
> ===================================================================
> RCS file: /cvs/ports/devel/highway/Makefile,v
> retrieving revision 1.11
> diff -u -p -u -p -r1.11 Makefile
> --- devel/highway/Makefile	25 Jul 2026 08:29:28 -0000	1.11
> +++ devel/highway/Makefile	4 Sep 2026 19:51:57 -0000
> @@ -4,7 +4,7 @@ GH_ACCOUNT =	google
>  GH_PROJECT =	highway
>  GH_TAGNAME =	1.4.0
>  
> -REVISION =	0
> +REVISION =	1
>  
>  CATEGORIES =	devel
>  
> @@ -26,13 +26,6 @@ WANTLIB +=	${COMPILER_LIBCXX} m
>  
>  CONFIGURE_ARGS +=	-DHWY_SYSTEM_GTEST=ON \
>  			-DBUILD_SHARED_LIBS=ON
> -
> -.if ${MACHINE_ARCH} == riscv64
> -# The riscv Vector code does not build because of a long vs long long mismatch
> -CXXFLAGS +=		-DHWY_BROKEN_RVV=HWY_RVV

> -# Don't assume risc-v Vector support
> -CONFIGURE_ARGS+=	-DHWY_CMAKE_RVV=OFF

This has to stay, we don't want to assume that the vector feature is
available.

I had another diff that was more complete.

> -.endif
>  
>  # everything except tests
>  ALL_TARGET =	${SHARED_LIBS:Mhwy*}
> 

-- 
jca