Mathieu Malaterre

@jan-wassenberg I just found out I cannot simply pass compile options to header file: * https://stackoverflow.com/questions/48162570/cmake-how-to-set-compile-flags-for-header-files instead I need to find all source file (cpp) which would need the special...

> OK :) But it's only a tiny change. We could add an opt-out CMake option for you that skips the rv64gcv1p0? (This would be used until runtime dispatch works,...

> I suspect we either want V in all files, or none - there's not much point to Highway nor its tests without some kind of vectors. The whole point...

> Hence the proposal to make it conditional on clang - this would continue to work for our use case, whereas for your GCC build it will disable V. In...

> Here is what's currently in CMakeLists: > > ``` > if(HWY_RISCV) > list(APPEND HWY_FLAGS -march=rv64gcv1p0) > if(${CMAKE_CXX_COMPILER_ID} MATCHES "Clang") > list(APPEND HWY_FLAGS -menable-experimental-extensions) > endif() > endif() > ```...

I believe I can reproduce the issue on Linux/i686/gcc-12: * https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106322

Something that is odd is that armel (32bits) is also affects (exact same symptoms), compare: * https://buildd.debian.org/status/fetch.php?pkg=highway&arch=i386&ver=0.17.1%7Egit20220711.f0a396a-1&stamp=1657558093&raw=0 and * https://buildd.debian.org/status/fetch.php?pkg=highway&arch=armel&ver=0.17.1%7Egit20220711.f0a396a-1&stamp=1657559820&raw=0

This has been fixed on GCC 12.x branch. I suspect we can close this bug: * https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106322#c48

> Thanks for following up 👍 Should we mark EMU128 as broken on GCC

The above patch has been integrated in debian package since 7/7/2022 with no failures neither on gcc-11 nor on gcc-12: * https://salsa.debian.org/debian-phototools-team/highway/-/blob/master/debian/patches/neon.patch Reference: SCALAR+NEON: * https://buildd.debian.org/status/fetch.php?pkg=highway&arch=armhf&ver=1.0.0-5&stamp=1659598256&raw=0 EMU128+NEON: * https://buildd.debian.org/status/fetch.php?pkg=highway&arch=armhf&ver=1.0.1%7Egit20220802.5810c58-4&stamp=1659618390&raw=0