[PATCH] gcc.c: Split up the driver's main into smaller functions

2014-10-03 Thread David Malcolm
The main function for the driver in gcc.c has grown from ~200 lines in its original form (way back in r262) to ~1000 lines today, with a dozen locals (if we include the params). The following patch splits it up into 15 smaller functions, moving the various locals into the places where they're

Re: [PATCH] gcc.c: Split up the driver's main into smaller functions

2014-10-03 Thread Joseph S. Myers
On Fri, 3 Oct 2014, David Malcolm wrote: The main function for the driver in gcc.c has grown from ~200 lines in its original form (way back in r262) to ~1000 lines today, with a dozen locals (if we include the params). The following patch splits it up into 15 smaller functions, moving the

[PATCH] Fix build on darwin (was Re: [PATCH] gcc.c: Split up the driver's main into smaller functions)

2014-10-03 Thread David Malcolm
On Fri, 2014-10-03 at 11:02 -0400, David Malcolm wrote: The main function for the driver in gcc.c has grown from ~200 lines in its original form (way back in r262) to ~1000 lines today, with a dozen locals (if we include the params). The following patch splits it up into 15 smaller

Re: [PATCH] Fix build on darwin (was Re: [PATCH] gcc.c: Split up the driver's main into smaller functions)

2014-10-03 Thread Iain Sandoe
Hi David, On 3 Oct 2014, at 20:15, David Malcolm wrote: On Fri, 2014-10-03 at 11:02 -0400, David Malcolm wrote: The main function for the driver in gcc.c has grown from ~200 lines in its original form (way back in r262) to ~1000 lines today, with a dozen locals (if we include the params).

Re: [PATCH] Fix build on darwin (was Re: [PATCH] gcc.c: Split up the driver's main into smaller functions)

2014-10-03 Thread Jeff Law
On 10/03/14 13:15, David Malcolm wrote: On Fri, 2014-10-03 at 11:02 -0400, David Malcolm wrote: The main function for the driver in gcc.c has grown from ~200 lines in its original form (way back in r262) to ~1000 lines today, with a dozen locals (if we include the params). The following patch