RFA: RL78: Add -fstack-usage support

RFA: RL78: Add -fstack-usage support

[Nick Clifton]

Hi DJ,

  Please could I apply the patch below to the RL78 port ?  It adds
  support for generating stack use info with the -fstack-usage option.

  Also - may I apply this patch to the 4.7 branch as well please ?

Cheers
  Nick

gcc/ChangeLog
2012-04-04  Nick Clifton  <nickc@redhat.com>

	* config/rl78/rl78.c (rl78_expand_prologue): Set stack use
	information, if requested.

Index: gcc/config/rl78/rl78.c
===================================================================
--- gcc/config/rl78/rl78.c	(revision 186130)
+++ gcc/config/rl78/rl78.c	(working copy)
@@ -827,6 +827,9 @@
   if (!cfun->machine->computed)
     rl78_compute_frame_info ();
 
+  if (flag_stack_usage)
+    current_function_static_stack_size = cfun->machine->framesize;
+
   for (i = 0; i < 16; i++)
     if (cfun->machine->need_to_push [i])
       {

Re: RFA: RL78: Add -fstack-usage support

[Eric Botcazou]

>   Please could I apply the patch below to the RL78 port ?  It adds
>   support for generating stack use info with the -fstack-usage option.

You probably need to adjust gcc.dg/stack-usage-1.c too.

> Index: gcc/config/rl78/rl78.c
> ===================================================================
> --- gcc/config/rl78/rl78.c	(revision 186130)
> +++ gcc/config/rl78/rl78.c	(working copy)
> @@ -827,6 +827,9 @@
>    if (!cfun->machine->computed)
>      rl78_compute_frame_info ();
>
> +  if (flag_stack_usage)
> +    current_function_static_stack_size = cfun->machine->framesize;
> +
>    for (i = 0; i < 16; i++)
>      if (cfun->machine->need_to_push [i])
>        {

s/flag_stack_usage/flag_stack_usage_info/

-- 
Eric Botcazou

Re: RFA: RL78: Add -fstack-usage support

[nick clifton]

[-- Attachment #1: Type: text/plain, Size: 530 bytes --]

Hi Eric,

 > On 04/04/12 12:24, Eric Botcazou wrote:
> You probably need to adjust gcc.dg/stack-usage-1.c too.
> s/flag_stack_usage/flag_stack_usage_info/

Thanks for the corrections.  Revised patch attached.

OK for mainline/4.7 branch ?

Cheers
   Nick

gcc/ChangeLog
2012-04-04  Nick Clifton  <nickc@redhat.com>

	* config/rl78/rl78.c (rl78_expand_prologue): Set stack use
	information, if requested.

gcc/testsuite/ChangeLog
2012-04-04  Nick Clifton  <nickc@redhat.com>

	* gcc.dg/stack-usage-1.c (SIZE): Define for the RL78.

[-- Attachment #2: rl78.stack-use.patch --]
[-- Type: text/x-patch, Size: 878 bytes --]

Index: gcc/config/rl78/rl78.c
===================================================================
--- gcc/config/rl78/rl78.c	(revision 186130)
+++ gcc/config/rl78/rl78.c	(working copy)
@@ -827,6 +827,9 @@
   if (!cfun->machine->computed)
     rl78_compute_frame_info ();
 
+  if (flag_stack_usage_info)
+    current_function_static_stack_size = cfun->machine->framesize;
+
   for (i = 0; i < 16; i++)
     if (cfun->machine->need_to_push [i])
       {
Index: gcc/testsuite/gcc.dg/stack-usage-1.c
===================================================================
--- gcc/testsuite/gcc.dg/stack-usage-1.c	(revision 186130)
+++ gcc/testsuite/gcc.dg/stack-usage-1.c	(working copy)
@@ -58,6 +58,8 @@
 #  define SIZE 224
 #elif defined (__epiphany__)
 #  define SIZE (256 - __EPIPHANY_STACK_OFFSET__)
+#elif defined (__RL78__)
+#  define SIZE 254
 #else
 #  define SIZE 256
 #endif

Re: RFA: RL78: Add -fstack-usage support

[DJ Delorie]

> OK for mainline/4.7 branch ?

Ok with me