[Valgrind-developers] [valgrind] s390: Fix compilation of testcases with clang 20.1.8

[Florian K. <fk...@so...>]

https://sourceware.org/cgit/valgrind/commit/?id=ddb16a98a91bf5ba2783f3276271e1412f7253b8

commit ddb16a98a91bf5ba2783f3276271e1412f7253b8
Author: Florian Krohm <fl...@ei...>
Date:   Sat Mar 21 18:04:25 2026 +0000

    s390: Fix compilation of testcases with clang 20.1.8
    
    bfp-fpc.c: The SRNMB insn requires the floating-point extension.
    Add -march=arch9 to compile flags which corresponds to z196 which is
    the minimum requires machine model.
    The clang error is apparently triggered by the mnemonic. C.f. bfp-convert.c
    which also uses insns requiring the floating-point extension. But those insns
    are pieced together in a different way avoiding the mnemonic.
    
    ex.c, exrl.c: The SGRK insn requires the distinct-operands facility.
    Add -march=arch9.
    
    vecdiv.c, vme.c: The insns require the vector facility.
    Add -march=z13.
    
    tmxx.c: The LOCGR insn requires the load/store-on-condition facility.
    Add -march=arch9.

Diff:
---
 memcheck/tests/s390x/Makefile.am | 2 ++
 none/tests/s390x/Makefile.am     | 4 ++++
 2 files changed, 6 insertions(+)

diff --git a/memcheck/tests/s390x/Makefile.am b/memcheck/tests/s390x/Makefile.am
index 19153bbb00..5994ebe362 100644
--- a/memcheck/tests/s390x/Makefile.am
+++ b/memcheck/tests/s390x/Makefile.am
@@ -20,3 +20,5 @@ vstrc_CFLAGS  = $(AM_CFLAGS) -march=z13
 vfae_CFLAGS   = $(AM_CFLAGS) -march=z13
 vistr_CFLAGS  = $(AM_CFLAGS) -march=z13
 vstrs_CFLAGS  = $(AM_CFLAGS) -march=z13
+vme_CFLAGS    = $(AM_CFLAGS) -march=z13
+tmxx_CFLAGS   = $(AM_CFLAGS) -march=arch9
diff --git a/none/tests/s390x/Makefile.am b/none/tests/s390x/Makefile.am
index ba127c85a2..56aa9f697e 100644
--- a/none/tests/s390x/Makefile.am
+++ b/none/tests/s390x/Makefile.am
@@ -81,8 +81,12 @@ vector_float_CFLAGS    = $(AM_CFLAGS) -march=z13 -DS390_TEST_COUNT=4
 vec2_CFLAGS      = $(AM_CFLAGS) -march=z13
 vec2_float_CFLAGS = $(AM_CFLAGS) -march=z13
 vec3_CFLAGS       = $(AM_CFLAGS) -march=z13
+vecdiv_CFLAGS     = $(AM_CFLAGS) -march=z13
 bfp_arith_CFLAGS   = $(AM_CFLAGS) @FLAG_MLONG_DOUBLE_128@
 bfp_compare_CFLAGS = $(AM_CFLAGS) @FLAG_MLONG_DOUBLE_128@
 bfp_convert_CFLAGS = $(AM_CFLAGS) @FLAG_MLONG_DOUBLE_128@
 bfp_load_CFLAGS    = $(AM_CFLAGS) @FLAG_MLONG_DOUBLE_128@
 bfp_tdc_CFLAGS     = $(AM_CFLAGS) @FLAG_MLONG_DOUBLE_128@
+bfp_fpc_CFLAGS     = $(AM_CFLAGS) -march=arch9
+ex_CFLAGS          = $(AM_CFLAGS) -march=arch9
+exrl_CFLAGS        = $(AM_CFLAGS) -march=arch9