function.c (stack_protect_epilogue): Use if rather than switch for check targetm...

2015-07-08  Kito Cheng  <kito.cheng@gmail.com>

	* function.c (stack_protect_epilogue): Use if rather than switch for
	check targetm.have_stack_protect_test.

From-SVN: r225599

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index c54e138220cda9b465cf1a2edec124834506577d..7747a59bed2f809ca2e1709edc857c50c949b09e 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2015-07-08  Kito Cheng  <kito.cheng@gmail.com>
+
+       * function.c (stack_protect_epilogue): Use if rather than switch for
+       check targetm.have_stack_protect_test.
+
 2015-07-08  Trevor Saunders  <tbsaunde+gcc@tbsaunde.org>
 
        * defaults.h: Provide default for WORD_REGISTER_OPERATIONS.

diff --git a/gcc/function.c b/gcc/function.c
index 972cdc87db08e8ea7adaa845c72f29c365309145..b87aef67989394ae83f2df1f29215eeaae28f4d4 100644 (file)
--- a/gcc/function.c
+++ b/gcc/function.c
@@ -4874,26 +4874,18 @@ stack_protect_epilogue (void)
   tree guard_decl = targetm.stack_protect_guard ();
   rtx_code_label *label = gen_label_rtx ();
   rtx x, y, tmp;
+  rtx_insn *seq;
 
   x = expand_normal (crtl->stack_protect_guard);
   y = expand_normal (guard_decl);
 
   /* Allow the target to compare Y with X without leaking either into
      a register.  */
-  switch (targetm.have_stack_protect_test ())
-    {
-    case 1:
-      if (rtx_insn *seq = targetm.gen_stack_protect_test (x, y, label))
-       {
-         emit_insn (seq);
-         break;
-       }
-      /* FALLTHRU */
-
-    default:
-      emit_cmp_and_jump_insns (x, y, EQ, NULL_RTX, ptr_mode, 1, label);
-      break;
-    }
+  if (targetm.have_stack_protect_test ()
+      && ((seq = targetm.gen_stack_protect_test (x, y, label)) != NULL_RTX))
+    emit_insn (seq);
+  else
+    emit_cmp_and_jump_insns (x, y, EQ, NULL_RTX, ptr_mode, 1, label);
 
   /* The noreturn predictor has been moved to the tree level.  The rtl-level
      predictors estimate this branch about 20%, which isn't enough to get