Re: [PATCH] Fix leaks in vectorizable_shift (PR middle-end/56461)

2013-03-04 Thread Richard Biener
On Fri, 1 Mar 2013, Jakub Jelinek wrote: > Hi! > > Except for the case when we set vec_oprnd1 to non-NULL, with j == 0 > we always initialize vec_oprnds[01] with vect_get_vec_defs, which overwrites > the vectors rather than just filling them. So we shouldn't allocate the > vectors first. > > Bo

[PATCH] Fix leaks in vectorizable_shift (PR middle-end/56461)

2013-03-01 Thread Jakub Jelinek
Hi! Except for the case when we set vec_oprnd1 to non-NULL, with j == 0 we always initialize vec_oprnds[01] with vect_get_vec_defs, which overwrites the vectors rather than just filling them. So we shouldn't allocate the vectors first. Bootstrapped/regtested on x86_64-linux and i686-linux, ok fo