Re: [PATCH] Rewrite lto streamer DFS from recursion to worklist (PR lto/65515)

2015-03-25 Thread Richard Biener
On Tue, 24 Mar 2015, Jakub Jelinek wrote: > On Tue, Mar 24, 2015 at 04:19:46PM +0100, Jakub Jelinek wrote: > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > Also tested with > ../configure --with-build-config=bootstrap-lto > --enable-languages=c,c++,fortran,objc,obj-c+

Re: [PATCH] Rewrite lto streamer DFS from recursion to worklist (PR lto/65515)

2015-03-24 Thread Jakub Jelinek
On Tue, Mar 24, 2015 at 04:19:46PM +0100, Jakub Jelinek wrote: > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? Also tested with ../configure --with-build-config=bootstrap-lto --enable-languages=c,c++,fortran,objc,obj-c++,go make -j16; make -j16 -k check on x86_64-linux, no

[PATCH] Rewrite lto streamer DFS from recursion to worklist (PR lto/65515)

2015-03-24 Thread Jakub Jelinek
Hi! Without this patch, compilation of limits-fndefn.c with -flto needs huge amounts of stack (more than 20 frames in backtrace). This patch reworks it so that we use a vector worklist instead, most of the DFS::DFS_write_tree function body has been moved into DFS::DFS and DFS_write_tree now ju