|
From: Ivo R. <ir...@so...> - 2017-09-27 05:35:27
|
https://sourceware.org/git/gitweb.cgi?p=valgrind.git;h=07e286ea303b2dfcb7188dcb6092b5025623b800 commit 07e286ea303b2dfcb7188dcb6092b5025623b800 Author: Ivo Raisr <iv...@iv...> Date: Tue Sep 26 09:33:27 2017 +0200 Reorder allocatable registers for s390x so that the callee saved are listed first. Helper calls always trash all caller saved registers. By listing the callee saved first then VEX register allocator (both v2 and v3) is more likely to pick them and does not need to spill that much before helper calls. Diff: --- VEX/priv/host_s390_defs.c | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/VEX/priv/host_s390_defs.c b/VEX/priv/host_s390_defs.c index ab2d0b2..327674a 100644 --- a/VEX/priv/host_s390_defs.c +++ b/VEX/priv/host_s390_defs.c @@ -397,20 +397,30 @@ getRRegUniverse_S390(void) fpr_index[i] = -1; /* Add the registers that are available to the register allocator. - GPRs: registers 1..11 are available - FPRs: registers 0..15 are available + GPRs: registers 6..11 are callee saved, list them first + registers 1..5 are caller saved, list them after + FPRs: registers 8..15 are callee saved, list them first + registers 0..7 are caller saved, list them after FPR12 - FPR15 are also used as register pairs for 128-bit floating point operations */ ru->allocable_start[HRcInt64] = ru->size; - for (UInt regno = 1; regno <= 11; ++regno) { + for (UInt regno = 6; regno <= 11; ++regno) { + gpr_index[regno] = ru->size; + ru->regs[ru->size++] = s390_hreg_gpr(regno); + } + for (UInt regno = 1; regno <= 5; ++regno) { gpr_index[regno] = ru->size; ru->regs[ru->size++] = s390_hreg_gpr(regno); } ru->allocable_end[HRcInt64] = ru->size - 1; ru->allocable_start[HRcFlt64] = ru->size; - for (UInt regno = 0; regno <= 15; ++regno) { + for (UInt regno = 8; regno <= 15; ++regno) { + fpr_index[regno] = ru->size; + ru->regs[ru->size++] = s390_hreg_fpr(regno); + } + for (UInt regno = 0; regno <= 7; ++regno) { fpr_index[regno] = ru->size; ru->regs[ru->size++] = s390_hreg_fpr(regno); } |