ppc_vsx-inl.h:6926:38: error: builtin ‘__vector(4) int __builtin_s390_vec_sum_u128(__vector(4) int, __vector(4) int)’ is deprecated [-Werror]

[malaterre]

@johnplatts I cannot compile highway 1.3.0 on s390x it fails with:

[100/225] /usr/bin/c++ -DHWY_SHARED_DEFINE -DTOOLCHAIN_MISS_ASM_HWCAP_H -I"/build/reproducible-path/highway-1.3.1~git20251202.ee36c83" -g -O2 -ffile-prefix-map=/build/reproducible-path/highway-1.3.1~git20251202.ee36c83=. -flto=auto -ffat-lto-objects -fstack-protector-strong -Wformat -Werror=format-security -DHWY_BROKEN_EMU128=0 -march=z14 -mzvector -Wdate-time -D_FORTIFY_SOURCE=2 -O3 -DNDEBUG -std=c++17 -fPIE -fvisibility=hidden -fvisibility-inlines-hidden -Wno-builtin-macro-redefined -D__DATE__=\"redacted\" -D__TIMESTAMP__=\"redacted\" -D__TIME__=\"redacted\" -fmerge-all-constants -Wall -Wextra -Wconversion -Wsign-conversion -Wvla -Wnon-virtual-dtor -Wcast-align -fmath-errno -fno-exceptions -Wno-psabi -Werror -DHWY_IS_TEST=1 -DGTEST_HAS_PTHREAD=1 -MD -MT CMakeFiles/reduction_test.dir/hwy/tests/reduction_test.cc.o -MF CMakeFiles/reduction_test.dir/hwy/tests/reduction_test.cc.o.d -o CMakeFiles/reduction_test.dir/hwy/tests/reduction_test.cc.o -c '/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/reduction_test.cc'
FAILED: [code=1] CMakeFiles/reduction_test.dir/hwy/tests/reduction_test.cc.o 
/usr/bin/c++ -DHWY_SHARED_DEFINE -DTOOLCHAIN_MISS_ASM_HWCAP_H -I"/build/reproducible-path/highway-1.3.1~git20251202.ee36c83" -g -O2 -ffile-prefix-map=/build/reproducible-path/highway-1.3.1~git20251202.ee36c83=. -flto=auto -ffat-lto-objects -fstack-protector-strong -Wformat -Werror=format-security -DHWY_BROKEN_EMU128=0 -march=z14 -mzvector -Wdate-time -D_FORTIFY_SOURCE=2 -O3 -DNDEBUG -std=c++17 -fPIE -fvisibility=hidden -fvisibility-inlines-hidden -Wno-builtin-macro-redefined -D__DATE__=\"redacted\" -D__TIMESTAMP__=\"redacted\" -D__TIME__=\"redacted\" -fmerge-all-constants -Wall -Wextra -Wconversion -Wsign-conversion -Wvla -Wnon-virtual-dtor -Wcast-align -fmath-errno -fno-exceptions -Wno-psabi -Werror -DHWY_IS_TEST=1 -DGTEST_HAS_PTHREAD=1 -MD -MT CMakeFiles/reduction_test.dir/hwy/tests/reduction_test.cc.o -MF CMakeFiles/reduction_test.dir/hwy/tests/reduction_test.cc.o.d -o CMakeFiles/reduction_test.dir/hwy/tests/reduction_test.cc.o -c '/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/reduction_test.cc'
In file included from /build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/highway.h:704,
                 from /build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/reduction_test.cc:24,
                 from /build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/foreach_target.h:312,
                 from /build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/reduction_test.cc:23:
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/ops/ppc_vsx-inl.h: In instantiation of ‘hwy::N_Z15::Vec128<T> hwy::N_Z15::detail::SumOfU32OrU64LanesAsU128(hwy::N_Z15::Vec128<T>) [with T = int; hwy::EnableIf<((bool)((! IsFloat<T>()) && (! IsSpecialFloat<T>())))>* <anonymous> = 0; hwy::EnableIf<(((long unsigned int{1} << sizeof (T)) & ((1 << 4) | (1 << 8))) != 0)>* <anonymous> = 0]’:
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/ops/ppc_vsx-inl.h:7095:55:   required from ‘hwy::N_Z15::VFromD<D> hwy::N_Z15::SumOfLanes(D, VFromD<D>) [with D = Simd<int, 4, 0>; hwy::EnableIf<((bool)((D::kPrivateLanes * sizeof (typename D::T)) == 16))>* <anonymous> = 0; hwy::EnableIf<((bool)IsSameEither<typename hwy::RemoveConstT<typename hwy::RemoveVolatileT<typename hwy::RemoveRefT<typename D::T>::type>::type>::type, unsigned int, int>())>* <anonymous> = 0; VFromD<D> = Vec128<int>]’
 7095 |   return Broadcast<3>(detail::SumOfU32OrU64LanesAsU128(v));
      |                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/reduction_test.cc:86:5:   required from ‘void hwy::N_Z15::{anonymous}::TestSumOfLanes::operator()(T, D) [with T = int; D = hwy::N_Z15::Simd<int, 4, 0>]’
   86 |     HWY_ASSERT_VEC_EQ(d, Set(d, sum), SumOfLanes(d, Load(d, in_lanes.get())));
      |                                       ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/test_util-inl.h:264:13:   required from ‘static void hwy::N_Z15::detail::ForeachCappedR<T, kMul, kMinArg, Test, kPow2>::Do(size_t, size_t) [with T = int; long unsigned int kMul = 4; long unsigned int kMinArg = 1; Test = hwy::N_Z15::{anonymous}::TestSumOfLanes; int kPow2 = 0; size_t = long unsigned int]’
  264 |       Test()(T(), d);
      |       ~~~~~~^~~~~~~~
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/test_util-inl.h:410:63:   required from ‘void hwy::N_Z15::ForExtendableVectors<Test, kPow2>::operator()(T) const [with T = int; Test = hwy::N_Z15::{anonymous}::TestSumOfLanes; int kPow2 = 0]’
  410 |     detail::ForeachCappedR<T, kMul, kMinArg, Test, -kPow2>::Do(1, max_lanes);
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/test_util-inl.h:634:36:   required from ‘void hwy::N_Z15::ForPartialVectors<Test>::operator()(T) const [with T = int; Test = hwy::N_Z15::{anonymous}::TestSumOfLanes]’
  634 |     ForExtendableVectors<Test, 0>()(t);
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/test_util-inl.h:681:7:   required from ‘void hwy::N_Z15::ForSignedTypes(const Func&) [with Func = ForPartialVectors<{anonymous}::TestSumOfLanes>]’
  681 |   func(int32_t());
      |   ~~~~^~~~~~~~~~~
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/test_util-inl.h:699:17:   required from ‘void hwy::N_Z15::ForIntegerTypes(const Func&) [with Func = ForPartialVectors<{anonymous}::TestSumOfLanes>]’
  699 |   ForSignedTypes(func);
      |   ~~~~~~~~~~~~~~^~~~~~
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/test_util-inl.h:767:18:   required from ‘void hwy::N_Z15::ForAllTypes(const Func&) [with Func = ForPartialVectors<{anonymous}::TestSumOfLanes>]’
  767 |   ForIntegerTypes(func);
      |   ~~~~~~~~~~~~~~~^~~~~~
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/tests/reduction_test.cc:103:14:   required from here
  103 |   ForAllTypes(ForPartialVectors<TestSumOfLanes>());
      |   ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/reproducible-path/highway-1.3.1~git20251202.ee36c83/hwy/ops/ppc_vsx-inl.h:6926:38: error: builtin ‘__vector(4) int __builtin_s390_vec_sum_u128(__vector(4) int, __vector(4) int)’ is deprecated [-Werror]
 6926 |       d, Vec128<uint8_t>{vec_sum_u128(BitCast(du, v).raw, Zero(du).raw)});
      |                                      ^

ref:

• https://buildd.debian.org/status/fetch.php?pkg=highway&arch=s390x&ver=1.3.1%7Egit20251202.ee36c83-1&stamp=1764888117&raw=0

Thanks for help !

[johnplatts]

@malaterre I fixed these compiler errors in pull request #2812, and there were some changes to Z14/Z15 compiler intrinsics in GCC 15 and Clang 20.

[malaterre]

Thanks @johnplatts ! I've uploaded the patch a minute ago:

• https://tracker.debian.org/news/1695879/accepted-highway-131git20251202ee36c83-2-source-into-experimental/

I'll keep you posted.

[malaterre]

LGTM:

• https://buildd.debian.org/status/fetch.php?pkg=highway&arch=s390x&ver=1.3.1%7Egit20251202.ee36c83-2&stamp=1765049180&raw=0

[jan-wassenberg]

Thank you @johnplatts for implementing the fix, and @malaterre for reporting/confirming!