On Fri, May 6, 2016 at 2:40 PM, Martin Liška <mli...@suse.cz> wrote:
> On 05/03/2016 11:07 AM, Bin.Cheng wrote:
>> Patch applied as suggested at r235808.
>>
>> Thanks,
>> bin
>
> Hi.
>
> Following patch introduces memory leak:
> /home/marxin/Programming/gcc2/objdir/gcc/xgcc 
> -B/home/marxin/Programming/gcc2/objdir/gcc/    -fno-diagnostics-show-caret 
> -fdiagnostics-color=never    -O3 -fomit-frame-pointer -funroll-loops 
> -fpeel-loops -ftracer -finline-functions  -w -c -o 920928-2.o 
> /home/marxin/Programming/gcc2/gcc/testsuite/gcc.c-torture/compile/920928-2.c
>
> ==5714== 40 bytes in 1 blocks are definitely lost in loss record 38 of 905
>
> ==5714==    at 0x4C2A00F: malloc (in 
> /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
>
> ==5714==    by 0x109C76F: xrealloc (xmalloc.c:178)
>
> ==5714==    by 0xA89696: reserve<edge_def*> (vec.h:288)
>
> ==5714==    by 0xA89696: reserve (vec.h:1438)
>
> ==5714==    by 0xA89696: safe_push (vec.h:1547)
>
> ==5714==    by 0xA89696: ifcvt_split_critical_edges (tree-if-conv.c:2397)
>
> ==5714==    by 0xA89696: tree_if_conversion (tree-if-conv.c:2725)
>
> ==5714==    by 0xA89696: (anonymous 
> namespace)::pass_if_conversion::execute(function*) (tree-if-conv.c:2832)
>
> ==5714==    by 0x98A9D3: execute_one_pass(opt_pass*) (passes.c:2348)
>
> ==5714==    by 0x98AF17: execute_pass_list_1(opt_pass*) [clone .constprop.84] 
> (passes.c:2432)
>
> ==5714==    by 0x98AF29: execute_pass_list_1(opt_pass*) [clone .constprop.84] 
> (passes.c:2433)
>
> ==5714==    by 0x98AF29: execute_pass_list_1(opt_pass*) [clone .constprop.84] 
> (passes.c:2433)
>
> ==5714==    by 0x98AF74: execute_pass_list(function*, opt_pass*) 
> (passes.c:2443)
>
> ==5714==    by 0x72DEB2: cgraph_node::expand() (cgraphunit.c:1982)
>
> ==5714==    by 0x72F3A3: expand_all_functions (cgraphunit.c:2118)
>
> ==5714==    by 0x72F3A3: symbol_table::compile() [clone .part.49] 
> (cgraphunit.c:2474)
>
> ==5714==    by 0x730D47: compile (cgraphunit.c:2538)
>
> ==5714==    by 0x730D47: symbol_table::finalize_compilation_unit() 
> (cgraphunit.c:2564)
>
> ==5714==    by 0xA392B7: compile_file() (toplev.c:488)
>
> ==5714==    by 0x616117: do_compile (toplev.c:1987)
>
> ==5714==    by 0x616117: toplev::main(int, char**) (toplev.c:2095)
>
> ==5714==    by 0x6182B6: main (main.c:39)
>
>
> Following patch fixes that, ready after it bootstraps and survives regtests?

Ok.

Richard.

> Thanks,
> Martin

Reply via email to