[PATCH] Fix PR65911

Markus Trippelsdorf markus@trippelsdorf.de
Tue Apr 28 10:34:00 GMT 2015


During the removal of "#if ARGS_GROW_DOWNWARD" in r222508, Trevor forgot
to add parentheses around a ternary operator.

Tested on powerpc64-unknown-linux-gnu.
Preapproved by Jakub and Richard on bugzilla. Commited to trunk.

2015-04-28  Markus Trippelsdorf  <markus@trippelsdorf.de>

	PR other/65911
	* function.c (pad_to_arg_alignment): Add parentheses.

diff --git a/gcc/function.c b/gcc/function.c
index d1a2c0a26b32..af4c087b51c1 100644
--- a/gcc/function.c
+++ b/gcc/function.c
@@ -4195,9 +4195,9 @@ pad_to_arg_alignment (struct args_size *offset_ptr, int boundary,
       else
 	{
 	  offset_ptr->constant = -sp_offset +
-	    ARGS_GROW_DOWNWARD ?
-	    FLOOR_ROUND (offset_ptr->constant + sp_offset, boundary_in_bytes) :
-	    CEIL_ROUND (offset_ptr->constant + sp_offset, boundary_in_bytes);
+	    (ARGS_GROW_DOWNWARD
+	    ? FLOOR_ROUND (offset_ptr->constant + sp_offset, boundary_in_bytes)
+	    : CEIL_ROUND (offset_ptr->constant + sp_offset, boundary_in_bytes));
 
 	    if (boundary > PARM_BOUNDARY)
 	      alignment_pad->constant = offset_ptr->constant - save_constant;
-- 
Markus



More information about the Gcc-patches mailing list