From: Richard Henderson Date: Wed, 19 Sep 2001 18:21:54 +0000 (-0700) Subject: * cfgbuild.c (find_sub_basic_blocks): Handle insns that can throw. X-Git-Tag: prereleases/libstdc++-3.0.95~2041 X-Git-Url: https://gcc.gnu.org/git/?a=commitdiff_plain;h=f068df3fe6fe7c87e74a999dca926594aae2c84b;p=gcc.git * cfgbuild.c (find_sub_basic_blocks): Handle insns that can throw. From-SVN: r45690 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 4375bacc157f..1ce093610ce9 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,7 @@ +2001-09-19 Richard Henderson + + * cfgbuild.c (find_sub_basic_blocks): Handle insns that can throw. + 2001-09-18 Catherine Moore * config/stormy16/stormy16.h (DEFAULT_PCC_STRUCT_RETURN): diff --git a/gcc/cfgbuild.c b/gcc/cfgbuild.c index 85185e562b34..d6e96e78d6c1 100644 --- a/gcc/cfgbuild.c +++ b/gcc/cfgbuild.c @@ -671,8 +671,8 @@ find_sub_basic_blocks (bb) { rtx insn = bb->head; rtx end = bb->end; - rtx jump_insn = NULL_RTX; - edge falltru = 0; + rtx flow_transfer_insn = NULL_RTX; + edge fallthru = NULL; basic_block first_bb = bb; int i; @@ -686,44 +686,66 @@ find_sub_basic_blocks (bb) while (1) { enum rtx_code code = GET_CODE (insn); + switch (code) { case BARRIER: - if (!jump_insn) + if (!flow_transfer_insn) abort (); break; + /* On code label, split current basic block. */ case CODE_LABEL: - falltru = split_block (bb, PREV_INSN (insn)); - if (jump_insn) - bb->end = jump_insn; - bb = falltru->dest; - remove_edge (falltru); - jump_insn = 0; + fallthru = split_block (bb, PREV_INSN (insn)); + if (flow_transfer_insn) + bb->end = flow_transfer_insn; + bb = fallthru->dest; + remove_edge (fallthru); + flow_transfer_insn = NULL_RTX; if (LABEL_ALTERNATE_NAME (insn)) make_edge (ENTRY_BLOCK_PTR, bb, 0); break; + case INSN: case JUMP_INSN: - /* In case we've previously split insn on the JUMP_INSN, move the - block header to proper place. */ - if (jump_insn) + case CALL_INSN: + /* In case we've previously split an insn that effects a control + flow transfer, move the block header to proper place. */ + if (flow_transfer_insn) { - falltru = split_block (bb, PREV_INSN (insn)); - bb->end = jump_insn; - bb = falltru->dest; - remove_edge (falltru); - jump_insn = 0; + fallthru = split_block (bb, PREV_INSN (insn)); + bb->end = flow_transfer_insn; + bb = fallthru->dest; + remove_edge (fallthru); + flow_transfer_insn = NULL_RTX; } + /* We need some special care for those expressions. */ - if (GET_CODE (insn) == JUMP_INSN) + if (code == JUMP_INSN) { if (GET_CODE (PATTERN (insn)) == ADDR_VEC || GET_CODE (PATTERN (insn)) == ADDR_DIFF_VEC) abort(); - jump_insn = insn; + flow_transfer_insn = insn; + } + else if (code == CALL_INSN) + { + rtx note; + if (nonlocal_goto_handler_labels + && (!(note = find_reg_note (insn, REG_EH_REGION, NULL_RTX)) + || INTVAL (XEXP (note, 0)) >= 0)) + flow_transfer_insn = insn; + else if (can_throw_internal (insn)) + flow_transfer_insn = insn; + else if (SIBLING_CALL_P (insn)) + flow_transfer_insn = insn; + else if (find_reg_note (insn, REG_NORETURN, 0)) + flow_transfer_insn = insn; } + else if (flag_non_call_exceptions && can_throw_internal (insn)) + flow_transfer_insn = insn; break; + default: break; } @@ -735,8 +757,8 @@ find_sub_basic_blocks (bb) /* In case expander replaced normal insn by sequence terminating by return and barrier, or possibly other sequence not behaving like ordinary jump, we need to take care and move basic block boundary. */ - if (jump_insn && GET_CODE (bb->end) != JUMP_INSN) - bb->end = jump_insn; + if (flow_transfer_insn) + bb->end = flow_transfer_insn; /* We've possibly replaced the conditional jump by conditional jump followed by cleanup at fallthru edge, so the outgoing edges may