]> git.ipfire.org Git - thirdparty/valgrind.git/commitdiff
Remove deprecated regression tests for mftgpr and mffgpr.
authorCarl Love <cel@us.ibm.com>
Fri, 10 Sep 2021 22:15:57 +0000 (17:15 -0500)
committerCarl Love <cel@us.ibm.com>
Mon, 13 Sep 2021 16:05:47 +0000 (11:05 -0500)
The mftgpr and mffgpr instructions are deprecated.  Added comments in
VEX/priv/guest_ppc_toIR.c for the instructions stating the instructions
are deprecated.  Valgrind support can be removed if the opcodes get reused
in the future.  For now, leaving the functional support in Valgrind
for the instructions.

Removed the regression test power6_mf_gpr.c, expect files and vgtest file
from none/tests/ppc64.

VEX/priv/guest_ppc_toIR.c
none/tests/ppc64/Makefile.am
none/tests/ppc64/power6_mf_gpr.c [deleted file]
none/tests/ppc64/power6_mf_gpr.stderr.exp [deleted file]
none/tests/ppc64/power6_mf_gpr.stdout.exp [deleted file]
none/tests/ppc64/power6_mf_gpr.vgtest [deleted file]

index 57ac7bcf481783c35eee195aa115d2b2c06d5ffa..e30e4315811fcd6321c960fde12f7cddc9383a22 100644 (file)
@@ -8401,6 +8401,9 @@ static Bool dis_int_logic ( UInt prefix, UInt theInstr )
          break;
 
       case 0x2DF: { // mftgpr (move floating-point to general purpose register)
+         /* The mftgpr instruction was deprecated in Power 7, 2009 timeframe.
+            Leaving support in Valgrind for now (9/10/2021).  Can remove the
+            mftgpr support in Valgrind if the opcode ever gets reused.  */
          IRTemp frB = newTemp(Ity_F64);
          DIP("mftgpr r%u,fr%u\n", rS_addr, rB_addr);
 
@@ -8415,6 +8418,9 @@ static Bool dis_int_logic ( UInt prefix, UInt theInstr )
       }
 
       case 0x25F: { // mffgpr (move floating-point from general purpose register)
+         /* The mffgpr instruction was deprecated in Power 7, 2009 timeframe.
+            Leaving support in Valgrind for now (9/10/2021).  Can remove the
+            mftgpr support in Valgrind if the opcode ever gets reused.  */
          IRTemp frA = newTemp(Ity_F64);
          DIP("mffgpr fr%u,r%u\n", rS_addr, rB_addr);
 
index df03dbec31d5c5675b21abf7e30e49004dd04a52..61b6ec93422b0d590aa82ed90a2865d555075662 100644 (file)
@@ -21,7 +21,6 @@ EXTRA_DIST = \
        tw_td.stderr.exp tw_td.stdout.exp tw_td.vgtest \
        opcodes.h \
        power6_bcmp.stderr.exp power6_bcmp.stdout.exp power6_bcmp.vgtest \
-       power6_mf_gpr.stderr.exp power6_mf_gpr.stdout.exp power6_mf_gpr.vgtest \
        test_isa_2_06_part1.stderr.exp  test_isa_2_06_part1.stdout.exp  test_isa_2_06_part1.vgtest \
        test_isa_2_06_part1.stdout.exp-LE \
        test_isa_2_06_part2.stderr.exp  test_isa_2_06_part2.stdout.exp  test_isa_2_06_part2.vgtest \
@@ -75,7 +74,7 @@ check_PROGRAMS = \
        test_isa_3_1_Misc test_isa_3_1_AT \
        subnormal_test test_darn_inst test_copy_paste \
        test_tm test_touch_tm data-cache-instructions \
-       power6_mf_gpr std_reg_imm \
+       std_reg_imm \
        twi_tdi tw_td power6_bcmp scv_test
 
 # lmw, stmw, lswi, lswx, stswi, stswx compile (and run) only on big endian.
diff --git a/none/tests/ppc64/power6_mf_gpr.c b/none/tests/ppc64/power6_mf_gpr.c
deleted file mode 100644 (file)
index 6151a34..0000000
+++ /dev/null
@@ -1,47 +0,0 @@
-/*  Copyright (C) 2007 IBM
-
-    Author: Pete Eberlein  eberlein@us.ibm.com
-
-    This program is free software; you can redistribute it and/or
-    modify it under the terms of the GNU General Public License as
-    published by the Free Software Foundation; either version 2 of the
-    License, or (at your option) any later version.
-
-    This program is distributed in the hope that it will be useful, but
-    WITHOUT ANY WARRANTY; without even the implied warranty of
-    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-    General Public License for more details.
-
-    You should have received a copy of the GNU General Public License
-    along with this program; if not, see <http://www.gnu.org/licenses/>.
-
-    The GNU General Public License is contained in the file COPYING.
-*/
-
-#include <stdio.h>
-#include <string.h>
-
-
-
-int main(int argc, char *argv[])
-{
-
-   long i;
-   double f;
-
-   i = 0;
-   f = 100.0;
-
-   printf("%lx %f\n", i, f);
-
- asm("mftgpr %0, %1\n": "=r"(i):"f"(f));
-
-   f = 0.0;
-   printf("%lx %f\n", i, f);
-
- asm("mffgpr %0, %1\n": "=f"(f):"r"(i));
-
-   printf("%lx %f\n", i, f);
-
-   return 0;
-}
diff --git a/none/tests/ppc64/power6_mf_gpr.stderr.exp b/none/tests/ppc64/power6_mf_gpr.stderr.exp
deleted file mode 100644 (file)
index 139597f..0000000
+++ /dev/null
@@ -1,2 +0,0 @@
-
-
diff --git a/none/tests/ppc64/power6_mf_gpr.stdout.exp b/none/tests/ppc64/power6_mf_gpr.stdout.exp
deleted file mode 100644 (file)
index 7bc827d..0000000
+++ /dev/null
@@ -1,3 +0,0 @@
-0 100.000000
-4059000000000000 0.000000
-4059000000000000 100.000000
diff --git a/none/tests/ppc64/power6_mf_gpr.vgtest b/none/tests/ppc64/power6_mf_gpr.vgtest
deleted file mode 100644 (file)
index 5868fe6..0000000
+++ /dev/null
@@ -1 +0,0 @@
-prog: power6_mf_gpr