[gc-improv] Convert ARM backend

[gc-improv] Convert ARM backend

[Laurynas Biveinis]

Bootstrapped and regtested under armv5tel-unknown-linux-gnueabi.
committed to gc-improv.

2009-11-09  Laurynas Biveinis  <laurynas.biveinis@gmail.com>

        * config/arm/arm.c (arm_init_machine_status): Use typed GC
        allocation.


-- 
Laurynas


Index: gcc/ChangeLog.gc-improv
===================================================================
--- gcc/ChangeLog.gc-improv     (revision 153879)
+++ gcc/ChangeLog.gc-improv     (working copy)
@@ -1,3 +1,8 @@
+2009-11-09  Laurynas Biveinis  <laurynas.biveinis@gmail.com>
+
+       * config/arm/arm.c (arm_init_machine_status): Use typed GC
+       allocation.
+
 2009-11-02  Laurynas Biveinis  <laurynas.biveinis@gmail.com>

        * sese.h: Include obstack.h.
Index: gcc/config/arm/arm.c
===================================================================
--- gcc/config/arm/arm.c        (revision 153879)
+++ gcc/config/arm/arm.c        (working copy)
@@ -18915,7 +18915,7 @@
 arm_init_machine_status (void)
 {
   struct machine_function *machine;
-  machine = (machine_function *) ggc_alloc_cleared (sizeof
(machine_function));
+  machine = ggc_alloc_cleared_machine_function();

 #if ARM_FT_UNKNOWN != 0
   machine->func_type = ARM_FT_UNKNOWN;

Re: [gc-improv] Convert ARM backend

[Daniel Jacobowitz]

On Mon, Nov 09, 2009 at 10:19:37AM +0100, Laurynas Biveinis wrote:
> -  machine = (machine_function *) ggc_alloc_cleared (sizeof
> (machine_function));
> +  machine = ggc_alloc_cleared_machine_function();

I happened to notice... you've lost the space before the open
parenthesis.  If you're doing this across a lot of backends, I figured
I'd point it out now before you tried to merge anything :-)

-- 
Daniel Jacobowitz
CodeSourcery

Re: [gc-improv] Convert ARM backend

[Laurynas Biveinis]

> On Mon, Nov 09, 2009 at 10:19:37AM +0100, Laurynas Biveinis wrote:
>> -  machine = (machine_function *) ggc_alloc_cleared (sizeof
>> (machine_function));
>> +  machine = ggc_alloc_cleared_machine_function();
>
> I happened to notice... you've lost the space before the open
> parenthesis.  If you're doing this across a lot of backends, I figured
> I'd point it out now before you tried to merge anything :-)

Thanks :) I am commiting this fix to gc-improv, tested by rebuilding arm.o.

2009-11-13  Laurynas Biveinis  <laurynas.biveinis@gmail.com>

        * config/arm/arm.c (arm_init_machine_status): Fix whitespace in
        the last commit.

Index: config/arm/arm.c
===================================================================
--- config/arm/arm.c    (revision 154027)
+++ config/arm/arm.c    (working copy)
@@ -18915,7 +18915,7 @@
 arm_init_machine_status (void)
 {
   struct machine_function *machine;
-  machine = ggc_alloc_cleared_machine_function();
+  machine = ggc_alloc_cleared_machine_function ();

 #if ARM_FT_UNKNOWN != 0
   machine->func_type = ARM_FT_UNKNOWN;



-- 
Laurynas