diff options
author | Richard Sandiford <rdsandiford@googlemail.com> | 2014-08-28 06:23:34 +0000 |
---|---|---|
committer | Richard Sandiford <rsandifo@gcc.gnu.org> | 2014-08-28 06:23:34 +0000 |
commit | bc702273a728f2840c2c9603926f0066e8e8cd3c (patch) | |
tree | 9838ef5790ac753d6dab4cc4bcc959abe4c9f6b1 /gcc | |
parent | 40954ce58f2f7b526e462228c6a08106bc9d058e (diff) | |
download | gcc-bc702273a728f2840c2c9603926f0066e8e8cd3c.zip gcc-bc702273a728f2840c2c9603926f0066e8e8cd3c.tar.gz gcc-bc702273a728f2840c2c9603926f0066e8e8cd3c.tar.bz2 |
jump.c: Include rtl-iter.h.
gcc/
* jump.c: Include rtl-iter.h.
(returnjump_p_1): Delete.
(returnjump_p): Use FOR_EACH_SUBRTX rather than for_each_rtx.
Remove handling of null rtxes.
From-SVN: r214643
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/ChangeLog | 7 | ||||
-rw-r--r-- | gcc/jump.c | 54 |
2 files changed, 32 insertions, 29 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 3447262..32f519d 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,12 @@ 2014-08-28 Richard Sandiford <rdsandiford@googlemail.com> + * jump.c: Include rtl-iter.h. + (returnjump_p_1): Delete. + (returnjump_p): Use FOR_EACH_SUBRTX rather than for_each_rtx. + Remove handling of null rtxes. + +2014-08-28 Richard Sandiford <rdsandiford@googlemail.com> + * ira.c: Include rtl-iter.h. (set_paradoxical_subreg): Turn from being a for_each_rtx callback to being a function that examines each subrtx itself. Remove @@ -54,6 +54,7 @@ along with GCC; see the file COPYING3. If not see #include "predict.h" #include "tree-pass.h" #include "target.h" +#include "rtl-iter.h" /* Optimize jump y; x: ... y: jumpif... x? Don't know if it is worth bothering with. */ @@ -68,7 +69,6 @@ static void mark_jump_label_1 (rtx, rtx, bool, bool); static void mark_jump_label_asm (rtx, rtx); static void redirect_exp_1 (rtx *, rtx, rtx, rtx); static int invert_exp_1 (rtx, rtx); -static int returnjump_p_1 (rtx *, void *); /* Worker for rebuild_jump_labels and rebuild_jump_labels_chain. */ static void @@ -920,39 +920,35 @@ condjump_label (const_rtx insn) return NULL_RTX; } -/* Return true if INSN is a (possibly conditional) return insn. */ - -static int -returnjump_p_1 (rtx *loc, void *data ATTRIBUTE_UNUSED) -{ - rtx x = *loc; - - if (x == NULL) - return false; - - switch (GET_CODE (x)) - { - case RETURN: - case SIMPLE_RETURN: - case EH_RETURN: - return true; - - case SET: - return SET_IS_RETURN_P (x); - - default: - return false; - } -} - /* Return TRUE if INSN is a return jump. */ int returnjump_p (rtx insn) { - if (!JUMP_P (insn)) - return 0; - return for_each_rtx (&PATTERN (insn), returnjump_p_1, NULL); + if (JUMP_P (insn)) + { + subrtx_iterator::array_type array; + FOR_EACH_SUBRTX (iter, array, PATTERN (insn), NONCONST) + { + const_rtx x = *iter; + switch (GET_CODE (x)) + { + case RETURN: + case SIMPLE_RETURN: + case EH_RETURN: + return true; + + case SET: + if (SET_IS_RETURN_P (x)) + return true; + break; + + default: + break; + } + } + } + return false; } /* Return true if INSN is a (possibly conditional) return insn. */ |