Re: [PATCH 1/6] Fix memory leak in cilk

On 11/27/2015 10:21 AM, Bernd Schmidt wrote: > Already improved, wasn't it? (Please don't quote the entire previous patch > next time). > > > Bernd Thanks, I just applied nits pointed by you. Patch installed as r231001. Thank you, Martin

Re: [PATCH 1/6] Fix memory leak in cilk

On 11/26/2015 09:59 PM, Martin Liška wrote: I'm sending v2 of the patch, where I removed adding of 'const' to certain function arguments. Apart from that, I found one more leak related to cilk. As I've retested in valgrind, there should not be any memory leak related to cilk. Ready to be

Re: [PATCH 1/6] Fix memory leak in cilk

C poison m_vec m_vecpfx m_vecdata #endif Hi. I'm sending v2 of the patch, where I removed adding of 'const' to certain function arguments. Apart from that, I found one more leak related to cilk. As I've retested in valgrind, there should not be any memory leak related to cilk. Ready to be installed?

Re: [PATCH 1/6] Fix memory leak in cilk

> diff --git a/gcc/c-family/cilk.c b/gcc/c-family/cilk.c > index e75e20c..1167b2b 100644 > --- a/gcc/c-family/cilk.c > +++ b/gcc/c-family/cilk.c > @@ -844,6 +844,7 @@ gimplify_cilk_spawn (tree *spawn_p) > call2, build_empty_stmt (EXPR_LOCATION (call1))); >

[PATCH 1/6] Fix memory leak in cilk

gcc/c/ChangeLog: 2015-11-20 Martin Liska PR c++/68312 * c-array-notation.c (fix_builtin_array_notation_fn): Use release_vec_vec instead of vec::release. (build_array_notation_expr): Likewise. (fix_conditional_array_notations_1):