[Valgrind-developers] [valgrind] lmw, lswi and related PowerPC insns aren't allowed on ppc64le

[Mark W. <ma...@so...>]

https://sourceware.org/git/gitweb.cgi?p=valgrind.git;h=74b74174d572fee4015b8f4e326db3cd949bcdc3

commit 74b74174d572fee4015b8f4e326db3cd949bcdc3
Author: Mark Wielaard <ma...@kl...>
Date:   Tue Dec 1 14:57:39 2020 +0100

    lmw, lswi and related PowerPC insns aren't allowed on ppc64le
    
    Newer binutils produce an error when the assembly contains lmw, stmw,
    lswi, lswx, stswi, or stswx instructions in little-endian mode.
    
    Only build and run the lsw and ldst_multiple testcases on ppc64[be].
    
    https://bugs.kde.org/show_bug.cgi?id=427870

Diff:
---
 NEWS                                  | 1 +
 none/tests/ppc64/Makefile.am          | 9 +++++++--
 none/tests/ppc64/ldst_multiple.vgtest | 1 +
 3 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/NEWS b/NEWS
index c26449a3e2..c9dfd73155 100644
--- a/NEWS
+++ b/NEWS
@@ -58,6 +58,7 @@ n-i-bz  helgrind: If hg_cli__realloc fails, return NULL.
 427400  PPC ISA 3.1 support is missing, part 4
 427401  PPC ISA 3.1 support is missing, part 5
 384729  __libc_freeres inhibits cross-platform valgrind
+427870  lmw, lswi and related PowerPC insns aren't allowed on ppc64le 
 427404  PPC ISA 3.1 support is missing, part 6
 429692  unhandled ppc64le-linux syscall: 147 (getsid)
 
diff --git a/none/tests/ppc64/Makefile.am b/none/tests/ppc64/Makefile.am
index a267b727a8..cd5dc91a46 100644
--- a/none/tests/ppc64/Makefile.am
+++ b/none/tests/ppc64/Makefile.am
@@ -59,7 +59,7 @@ EXTRA_DIST = \
 
 check_PROGRAMS = \
 	allexec \
-	lsw jm-insns round \
+	jm-insns round \
 	test_isa_2_06_part1 test_isa_2_06_part2 test_isa_2_06_part3 \
 	test_dfp1 test_dfp2 test_dfp3 test_dfp4 test_dfp5 \
 	test_isa_2_07_part1 test_isa_2_07_part2 \
@@ -67,10 +67,15 @@ check_PROGRAMS = \
 	test_isa_3_1_RT test_isa_3_1_XT test_isa_3_1_VRT \
 	test_isa_3_1_Misc test_isa_3_1_AT \
 	subnormal_test \
-	test_tm test_touch_tm ldst_multiple data-cache-instructions \
+	test_tm test_touch_tm data-cache-instructions \
 	power6_mf_gpr std_reg_imm \
 	twi_tdi tw_td power6_bcmp
 
+# lmw, stmw, lswi, lswx, stswi, stswx compile (and run) only on big endian.
+if VGCONF_PLATFORMS_INCLUDE_PPC64BE_LINUX
+check_PROGRAMS += lsw ldst_multiple
+endif
+
 AM_CFLAGS    += @FLAG_M64@
 AM_CXXFLAGS  += @FLAG_M64@
 AM_CCASFLAGS += @FLAG_M64@
diff --git a/none/tests/ppc64/ldst_multiple.vgtest b/none/tests/ppc64/ldst_multiple.vgtest
index 87e668e09f..22dd46c993 100644
--- a/none/tests/ppc64/ldst_multiple.vgtest
+++ b/none/tests/ppc64/ldst_multiple.vgtest
@@ -1 +1,2 @@
+prereq: ../../../tests/is_ppc64_BE
 prog: ldst_multiple