Re: [PATCH] Fix vectorizable_store

2013-06-27 Thread Richard Biener
Jakub Jelinek wrote: >Hi! > >On the gomp4 branch I've noticed a miscompilation of the simd-3.C >testcase I'm adding there, but even say >int a[1024] __attribute__((aligned (32))) = { 1 }; >int b[1024] __attribute__((aligned (32))) = { 1 }; >unsigned short c[1024] __attribute__((aligned (32))) = {

[PATCH] Fix vectorizable_store

2013-06-26 Thread Jakub Jelinek
Hi! On the gomp4 branch I've noticed a miscompilation of the simd-3.C testcase I'm adding there, but even say int a[1024] __attribute__((aligned (32))) = { 1 }; int b[1024] __attribute__((aligned (32))) = { 1 }; unsigned short c[1024] __attribute__((aligned (32))) = { 1 }; __attribute__((noinline

Re: [PATCH] Fix vectorizable_store memory leaks (PR middle-end/56461)

2013-03-04 Thread Richard Biener
On Fri, 1 Mar 2013, Jakub Jelinek wrote: > Hi! > > The first two hunks are similar issue to what I've posted for > vect_permute_load_chain two days ago, the remaining issue is that > if ncopies > 1, we'd leak the result_chain vector too. > > Fixed thusly, bootstrapped/regtested on x86_64-linux a

Re: [PATCH] Fix vectorizable_store memory leaks (PR middle-end/56461)

2013-03-01 Thread Jeff Law
On 03/01/2013 01:23 PM, Jakub Jelinek wrote: Hi! The first two hunks are similar issue to what I've posted for vect_permute_load_chain two days ago, the remaining issue is that if ncopies > 1, we'd leak the result_chain vector too. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-l

[PATCH] Fix vectorizable_store memory leaks (PR middle-end/56461)

2013-03-01 Thread Jakub Jelinek
Hi! The first two hunks are similar issue to what I've posted for vect_permute_load_chain two days ago, the remaining issue is that if ncopies > 1, we'd leak the result_chain vector too. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2013-03-01 Jakub Jelinek