[27/50] jump.c:eh_returnjump_p
diff mbox

Message ID 87ppgh8xxm.fsf@googlemail.com
State New
Headers show

Commit Message

Richard Sandiford Aug. 3, 2014, 2:12 p.m. UTC
gcc/
	* jump.c (eh_returnjump_p_1): Delete.
	(eh_returnjump_p): Use FOR_EACH_SUBRTX rather than for_each_rtx.
	Remove handling of null rtxes.

Comments

Jeff Law Aug. 5, 2014, 9:11 p.m. UTC | #1
On 08/03/14 08:12, Richard Sandiford wrote:
> gcc/
> 	* jump.c (eh_returnjump_p_1): Delete.
> 	(eh_returnjump_p): Use FOR_EACH_SUBRTX rather than for_each_rtx.
> 	Remove handling of null rtxes.
OK.
jeff

Patch
diff mbox

Index: gcc/jump.c
===================================================================
--- gcc/jump.c	2014-08-03 11:25:27.442127486 +0100
+++ gcc/jump.c	2014-08-03 11:25:27.722130254 +0100
@@ -953,18 +953,17 @@  returnjump_p (rtx insn)
 
 /* Return true if INSN is a (possibly conditional) return insn.  */
 
-static int
-eh_returnjump_p_1 (rtx *loc, void *data ATTRIBUTE_UNUSED)
-{
-  return *loc && GET_CODE (*loc) == EH_RETURN;
-}
-
 int
 eh_returnjump_p (rtx insn)
 {
-  if (!JUMP_P (insn))
-    return 0;
-  return for_each_rtx (&PATTERN (insn), eh_returnjump_p_1, NULL);
+  if (JUMP_P (insn))
+    {
+      subrtx_iterator::array_type array;
+      FOR_EACH_SUBRTX (iter, array, PATTERN (insn), NONCONST)
+	if (GET_CODE (*iter) == EH_RETURN)
+	  return true;
+    }
+  return false;
 }
 
 /* Return true if INSN is a jump that only transfers control and