Pointed out in IRC.   The intent as well as the docs and prototype had
the return type as a bool, but the default implementation used
HOST_WIDE_INT.  Opps.

This fixes the default implementation to match the intent, docs &
prototype.  Bootstrapped and regression tested.  Installing on the trunk
as obvious.

Jeff
commit fdc63db3b0343eaa3cdde6d5c143fbd80a851da6
Author: Jeff Law <l...@torsion.usersys.redhat.com>
Date:   Mon Oct 9 17:29:58 2017 -0400

            * targhooks.c (default_stack_clash_protection_final_dynamic_probe): 
Fix
            return type to match prototype and documentation.

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 202dd89b957..30bbeb5ad82 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2017-10-09  Jeff Law  <l...@redhat.com>
+
+       * targhooks.c (default_stack_clash_protection_final_dynamic_probe): Fix
+       return type to match prototype and documentation.
+
 2010-10-09  Segher Boessenkool  <seg...@kernel.crashing.org>
 
        * config/rs6000/rs6000.c (processor_costs): Move to ...
diff --git a/gcc/targhooks.c b/gcc/targhooks.c
index d56f53d35d8..8dbf8a14970 100644
--- a/gcc/targhooks.c
+++ b/gcc/targhooks.c
@@ -2235,7 +2235,7 @@ default_excess_precision (enum excess_precision_type 
ATTRIBUTE_UNUSED)
   return FLT_EVAL_METHOD_PROMOTE_TO_FLOAT;
 }
 
-HOST_WIDE_INT
+bool
 default_stack_clash_protection_final_dynamic_probe (rtx residual 
ATTRIBUTE_UNUSED)
 {
   return 0;

Reply via email to