[backport from gcc-4.9/trunk r197037 ] List-Archive: From: Eric Botcazou Subject: Fix bug in reload_combine with simple returns Date: Mon, 25 Mar 2013 12:25:33 +0100 Hi, another bug discovered for a cc0 target with conditional returns: the special code in reload_combine dealing with returns hasn't been updated for simple returns. Fixed thusly, tested on x86_64-suse-linux, applied on the mainline. gcc/ 2013-03-25 Eric Botcazou * postreload.c (reload_combine): Fix code detecting returns. --- gcc-4.8.0/gcc/postreload.c.~1~ 2013-01-10 21:38:27.000000000 +0100 +++ gcc-4.8.0/gcc/postreload.c 2013-03-29 14:07:07.452424219 +0100 @@ -1387,7 +1387,7 @@ reload_combine (void) } } - if (control_flow_insn && GET_CODE (PATTERN (insn)) != RETURN) + if (control_flow_insn && !ANY_RETURN_P (PATTERN (insn))) { /* Non-spill registers might be used at the call destination in some unknown fashion, so we have to mark the unknown use. */ @@ -1395,13 +1395,19 @@ reload_combine (void) if ((condjump_p (insn) || condjump_in_parallel_p (insn)) && JUMP_LABEL (insn)) - live = &LABEL_LIVE (JUMP_LABEL (insn)); + { + if (ANY_RETURN_P (JUMP_LABEL (insn))) + live = NULL; + else + live = &LABEL_LIVE (JUMP_LABEL (insn)); + } else live = &ever_live_at_start; - for (r = 0; r < FIRST_PSEUDO_REGISTER; r++) - if (TEST_HARD_REG_BIT (*live, r)) - reg_state[r].use_index = -1; + if (live) + for (r = 0; r < FIRST_PSEUDO_REGISTER; r++) + if (TEST_HARD_REG_BIT (*live, r)) + reg_state[r].use_index = -1; } reload_combine_note_use (&PATTERN (insn), insn, reload_combine_ruid,