]> sourceware.org Git - valgrind.git/commit
PowerPC:, Update test test_isa_3_1_R1_RT.c, test_isa_3_1_R1_XT.c
authorCarl Love <cel@us.ibm.com>
Wed, 19 Apr 2023 17:45:19 +0000 (13:45 -0400)
committerCarl Love <cel@us.ibm.com>
Wed, 19 Apr 2023 18:43:48 +0000 (14:43 -0400)
commit19c9e2418cf7aed6a1ce9f17ff3a2adcb877390c
treee4cb9b5b4f71e160d914f6726e02b4b60f1af177
parentb0e9fef2019e8ccc04bfa012077e41fc5e41f5b4
PowerPC:, Update test test_isa_3_1_R1_RT.c, test_isa_3_1_R1_XT.c

The commit:

  commit 20cc0680c3491e062c76605b24e76dc02e16ef47
  Author: Carl Love <cel@us.ibm.com>
  Date:   Mon Apr 17 17:12:25 2023 -0400

      PowerPC:, Fix test test_isa_3_1_R1_RT.c, test_isa_3_1_R1_XT.c

Fixes an issue with the PAD_ORI used in the the tests by explicitly adding
SAVE_REGS and RESTORE_REGS macros.  The macros ensure that the block of
immediate OR instructions don't inadvertently change the contents of the
registers.

John Reiser suggested that the PAD_ORI asm statements in the PAD_ORI
macro be updated to inform the compiler which register the ori instruction
is clobbering.  The compiler will then generate the code to save and
restore the register automatically.  This is a cleaner solution then
explicitly adding the macros to store and restore the registers.  It is
functionally cleaner in that the value fetched by the instruction under
test is not modified by the PAD_ORI instructions.

This patch removes the SAVE_REG and RESTORE_REG macros and updates the
PAD_ORI macro.
none/tests/ppc64/isa_3_1_helpers.h
none/tests/ppc64/test_isa_3_1_R1_RT.c
none/tests/ppc64/test_isa_3_1_R1_RT.stdout.exp
none/tests/ppc64/test_isa_3_1_R1_XT.c
none/tests/ppc64/test_isa_3_1_R1_XT.stdout.exp
This page took 0.033761 seconds and 5 git commands to generate.