Re: Clean up pretty printers [18/n]

2013-08-26 Thread Jan Hubicka
> On Mon, Aug 26, 2013 at 9:32 AM, Jan Hubicka wrote: > > >> > >> modify ALL_LINKERFLAGS or BUILD_LINKERFLAGS? > > > > configure --with-build-config=bootstrap-lto . You need plugin enabled > > buinutils for that. > > I think there however should be a lot easier way to reproduce it by just > > d

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 9:32 AM, Jan Hubicka wrote: >> >> modify ALL_LINKERFLAGS or BUILD_LINKERFLAGS? > > configure --with-build-config=bootstrap-lto . You need plugin enabled > buinutils for that. > I think there however should be a lot easier way to reproduce it by just > dropping some reall

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 9:32 AM, Jan Hubicka wrote: >> On Mon, Aug 26, 2013 at 9:00 AM, Jan Hubicka wrote: >> >> On Mon, Aug 26, 2013 at 7:24 AM, Jan Hubicka wrote: >> >> > Hi, >> >> > it seems to be couple weeks I am not able to compile big ltrans unit >> >> > with -fdump-tree-all >> >> > beca

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Jan Hubicka
> On Mon, Aug 26, 2013 at 9:00 AM, Jan Hubicka wrote: > >> On Mon, Aug 26, 2013 at 7:24 AM, Jan Hubicka wrote: > >> > Hi, > >> > it seems to be couple weeks I am not able to compile big ltrans unit > >> > with -fdump-tree-all > >> > because the compiler eventually runs out of memory. I think it

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 9:00 AM, Jan Hubicka wrote: >> On Mon, Aug 26, 2013 at 7:24 AM, Jan Hubicka wrote: >> > Hi, >> > it seems to be couple weeks I am not able to compile big ltrans unit with >> > -fdump-tree-all >> > because the compiler eventually runs out of memory. I think it is one of

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Jan Hubicka
> On Mon, Aug 26, 2013 at 7:24 AM, Jan Hubicka wrote: > > Hi, > > it seems to be couple weeks I am not able to compile big ltrans unit with > > -fdump-tree-all > > because the compiler eventually runs out of memory. I think it is one of > > your patches > > introducing serious memory leak. Can

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
On Mon, Aug 26, 2013 at 7:24 AM, Jan Hubicka wrote: > Hi, > it seems to be couple weeks I am not able to compile big ltrans unit with > -fdump-tree-all > because the compiler eventually runs out of memory. I think it is one of > your patches > introducing serious memory leak. Can you, please,

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Jan Hubicka
Hi, it seems to be couple weeks I am not able to compile big ltrans unit with -fdump-tree-all because the compiler eventually runs out of memory. I think it is one of your patches introducing serious memory leak. Can you, please, take a look on this? It makes my life harder - it is not fun to

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Paolo Carlini
On 08/26/2013 11:20 AM, Gabriel Dos Reis wrote: You need more than that. The rest of patch is in trunk now. Sorry for the breakage -- as you probably guessed, I specified only c-family directory on the commit command line. Thanks! Paolo.

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
Paolo Carlini writes: | Hi Gaby, | | bootstrap is currently broken: | | /scratch/Gcc/svn-dirs/trunk/gcc/c/c-objc-common.c: In function ‘bool | c_tree_printer(pretty_printer*, text_info*, const char*, int, bool, | bool, bool)’: | /scratch/Gcc/svn-dirs/trunk/gcc/c/c-objc-common.c:123:31: error: |

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
Paolo Carlini writes: | Hi Gaby. | | On 08/26/2013 10:42 AM, Gabriel Dos Reis wrote: | > Same topic as patch 17/n. For more expressing printers. | > Tested on an x86_64-suse-linux. Applied to mainline. | Just got this: | | /scratch/Gcc/svn-dirs/trunk/gcc/c/c-objc-common.c: In function ‘bool |

Re: Clean up pretty printers [18/n]

2013-08-26 Thread Paolo Carlini
Hi Gaby, bootstrap is currently broken: /scratch/Gcc/svn-dirs/trunk/gcc/c/c-objc-common.c: In function ‘bool c_tree_printer(pretty_printer*, text_info*, const char*, int, bool, bool, bool)’: /scratch/Gcc/svn-dirs/trunk/gcc/c/c-objc-common.c:123:31: error: ‘pp_c_expression’ was not declared in

Clean up pretty printers [18/n]

2013-08-26 Thread Gabriel Dos Reis
Same topic as patch 17/n. For more expressing printers. Tested on an x86_64-suse-linux. Applied to mainline. -- Gaby 2013-08-26 Gabriel Dos Reis c-family/ * c-pretty-print.h (c_pretty_printer::unary_expression): Now a virtual member function. (c_pretty_printer::mult