On Fri, Dec 26, 2025 at 06:55:14AM +0100, Rafael Sadowski wrote:
> On Fri Dec 26, 2025 at 10:16:46AM +1100, Jonathan Gray wrote:
> > On Thu, Dec 25, 2025 at 11:57:16PM +0100, Peter Hessler wrote:
> > > 
> > > 
> > > :http://build-failures.rhaalovely.net/aarch64/2025-12-22/x11/qt6/qtwebengine.log
> > > :
> > > 
> > > In file included from ../../../../../qtwebengine-everywhere-src-6.10.1/src/3rdparty/chromium/third_party/libaom/source/libaom/av1/encoder/arm/av1_error_sve.c:19:
> > > ../../../../../qtwebengine-everywhere-src-6.10.1/src/3rdparty/chromium/third_party/libaom/source/libaom/aom_dsp/arm/aom_neon_sve_bridge.h:15:10:
> > > fatal error: 'arm_neon_sve_bridge.h' file not found
> > >    15 | #include <arm_neon_sve_bridge.h>
> > >       |          ^~~~~~~~~~~~~~~~~~~~~~~
> > 
> > The log shows base clang.
> > 
> > Does installing the include fix the build?
> 
> I tried to mix the headers from base and ports to get access to
> arm_neon_sve_bridge without luck. I would be grateful if we could do it
> that way. CC robert@

FWIW I fixed Robert's mail address.  A similar diff was already
proposed by Brad:

  https://marc.info/?l=openbsd-tech&m=174155733400782&w=2

But it resulted in build failures using clang-19:

  https://marc.info/?l=openbsd-tech&m=174482865617988&w=2

For this header addition to be a clear improvement we'd need to first
apply Brad's proposal which backports some code from llvm/20:

  https://marc.info/?l=openbsd-ports&m=174555990331066&w=2

Regarding the diff below,

> > 
> > Index: gnu/usr.bin/clang/include/clang/intrin/Makefile
> > ===================================================================
> > RCS file: /cvs/src/gnu/usr.bin/clang/include/clang/intrin/Makefile,v
> > diff -u -p -r1.29 Makefile
> > --- gnu/usr.bin/clang/include/clang/intrin/Makefile	13 Jun 2025 17:11:09 -0000	1.29
> > +++ gnu/usr.bin/clang/include/clang/intrin/Makefile	25 Dec 2025 23:05:51 -0000
> > @@ -30,7 +30,7 @@ GEN+=		arm_bf16.h
> >  GEN+=		arm_mve.h
> >  GEN+=		arm_cde.h
> >  GEN+=		arm_vector_types.h
> > -HEADERS+=	arm_acle.h arm_cmse.h ${GEN}
> > +HEADERS+=	arm_acle.h arm_cmse.h arm_neon_sve_bridge.h ${GEN}

Brad's diff only installed the header on aarch64.
gnu/llvm/clang/lib/Headers/CMakeLists.txt indeed marks this file as
"aarch64-only" so I'd suggest we use Brad's diff.

> >  .elif ${MACHINE_ARCH} == "amd64" || ${MACHINE_ARCH} == "i386"
> >  HEADERS+=	__wmmintrin_aes.h \
> >  		__wmmintrin_pclmul.h \

-- 
jca