On 2025 Dec 26 (Fri) at 18:00:50 +0100 (+0100), Jeremie Courreges-Anglas wrote:
: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.
:
Does someone have a complete diff I can test? I'm not sure what part
from ports and what parts from src I need to use.
:> > .elif ${MACHINE_ARCH} == "amd64" || ${MACHINE_ARCH} == "i386"
:> > HEADERS+= __wmmintrin_aes.h \
:> > __wmmintrin_pclmul.h \
:
:--
:jca
:
