diff options
author | law <law@138bc75d-0d04-0410-961f-82ee72b054a4> | 1999-01-17 18:26:42 +0000 |
---|---|---|
committer | law <law@138bc75d-0d04-0410-961f-82ee72b054a4> | 1999-01-17 18:26:42 +0000 |
commit | 908d57eea99f3537d001f57ef5237559348f056d (patch) | |
tree | bbf24369179c5d4521bdd3c753d4bba4a116fe2d /gcc/regmove.c | |
parent | 4ea6f5119498af3294c3709a4d8cd081f35242b7 (diff) | |
download | ppe42-gcc-908d57eea99f3537d001f57ef5237559348f056d.tar.gz ppe42-gcc-908d57eea99f3537d001f57ef5237559348f056d.zip |
* regmove.c (optimize_reg_copy_1): Undo Aug 18 change. Update
REG_N_CALLS_CROSSED and REG_LIVE_LENGH if and only if we change
where a register is live.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@24718 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'gcc/regmove.c')
-rw-r--r-- | gcc/regmove.c | 76 |
1 files changed, 27 insertions, 49 deletions
diff --git a/gcc/regmove.c b/gcc/regmove.c index 87de0fb9808..9697fa23dc8 100644 --- a/gcc/regmove.c +++ b/gcc/regmove.c @@ -253,10 +253,10 @@ optimize_reg_copy_1 (insn, dest, src) && GET_MODE (XEXP (note, 0)) == GET_MODE (src)) { int failed = 0; - int length = 0; int d_length = 0; - int n_calls = 0; + int s_length = 0; int d_n_calls = 0; + int s_n_calls = 0; /* We can do the optimization. Scan forward from INSN again, replacing regs as we go. Set FAILED if a replacement can't @@ -290,42 +290,14 @@ optimize_reg_copy_1 (insn, dest, src) insn in the REG_N_REFS updates below. If this is not correct, no great harm is done. - - We do not undo this substitution if something later - fails. Therefore, we must update the other REG_N_* - counters now to keep them accurate. */ + Since we do not know if we will change the lifetime of + SREGNO or DREGNO, we must not update REG_LIVE_LENGTH + or REG_N_CALLS_CROSSED at this time. */ if (sregno >= FIRST_PSEUDO_REGISTER) - { - REG_N_REFS (sregno) -= loop_depth; - - if (REG_LIVE_LENGTH (sregno) >= 0) - { - REG_LIVE_LENGTH (sregno) -= length; - /* REG_LIVE_LENGTH is only an approximation after - combine if sched is not run, so make sure that - we still have a reasonable value. */ - if (REG_LIVE_LENGTH (sregno) < 2) - REG_LIVE_LENGTH (sregno) = 2; - } - - REG_N_CALLS_CROSSED (sregno) -= n_calls; - } + REG_N_REFS (sregno) -= loop_depth; if (dregno >= FIRST_PSEUDO_REGISTER) - { - REG_N_REFS (dregno) += loop_depth; - - if (REG_LIVE_LENGTH (dregno) >= 0) - REG_LIVE_LENGTH (dregno) += d_length; - - REG_N_CALLS_CROSSED (dregno) += d_n_calls; - } - - /* We've done a substitution, clear the counters. */ - length = 0; - d_length = 0; - n_calls = 0; - d_n_calls = 0; + REG_N_REFS (dregno) += loop_depth; } else { @@ -334,9 +306,10 @@ optimize_reg_copy_1 (insn, dest, src) } } - /* Count the insns and CALL_INSNs passed. If we passed the - death note of DEST, show increased live length. */ - length++; + /* For SREGNO, count the total number of insns scanned. + For DREGNO, count the total number of insns scanned after + passing the death note for DREGNO. */ + s_length++; if (dest_death) d_length++; @@ -344,7 +317,9 @@ optimize_reg_copy_1 (insn, dest, src) as a call that has been crossed. Otherwise, count it. */ if (q != p && GET_CODE (q) == CALL_INSN) { - n_calls++; + /* Similarly, total calls for SREGNO, total calls beyond + the death note for DREGNO. */ + s_n_calls++; if (dest_death) d_n_calls++; } @@ -364,11 +339,13 @@ optimize_reg_copy_1 (insn, dest, src) if (! failed) { + /* These counters need to be updated if and only if we are + going to move the REG_DEAD note. */ if (sregno >= FIRST_PSEUDO_REGISTER) { if (REG_LIVE_LENGTH (sregno) >= 0) { - REG_LIVE_LENGTH (sregno) -= length; + REG_LIVE_LENGTH (sregno) -= s_length; /* REG_LIVE_LENGTH is only an approximation after combine if sched is not run, so make sure that we still have a reasonable value. */ @@ -376,15 +353,7 @@ optimize_reg_copy_1 (insn, dest, src) REG_LIVE_LENGTH (sregno) = 2; } - REG_N_CALLS_CROSSED (sregno) -= n_calls; - } - - if (dregno >= FIRST_PSEUDO_REGISTER) - { - if (REG_LIVE_LENGTH (dregno) >= 0) - REG_LIVE_LENGTH (dregno) += d_length; - - REG_N_CALLS_CROSSED (dregno) += d_n_calls; + REG_N_CALLS_CROSSED (sregno) -= s_n_calls; } /* Move death note of SRC from P to INSN. */ @@ -398,6 +367,15 @@ optimize_reg_copy_1 (insn, dest, src) { XEXP (dest_death, 1) = REG_NOTES (p); REG_NOTES (p) = dest_death; + + if (dregno >= FIRST_PSEUDO_REGISTER) + { + /* If and only if we are moving the death note for DREGNO, + then we need to update its counters. */ + if (REG_LIVE_LENGTH (dregno) >= 0) + REG_LIVE_LENGTH (dregno) += d_length; + REG_N_CALLS_CROSSED (dregno) += d_n_calls; + } } return ! failed; |