Re: [PATCH] Handle vectorization of invariant loads (PR46787)

2011-07-28 Thread H.J. Lu
On Mon, Jul 4, 2011 at 6:28 AM, H.J. Lu wrote: > On Wed, Jun 29, 2011 at 4:19 AM, Richard Guenther wrote: >> >> The following patch makes us handle invariant loads during vectorization. >> Dependence analysis currently isn't clever enough to disambiguate them >> thus we insert versioning-for-alia

Re: [PATCH] Handle vectorization of invariant loads (PR46787)

2011-07-04 Thread H.J. Lu
On Wed, Jun 29, 2011 at 4:19 AM, Richard Guenther wrote: > > The following patch makes us handle invariant loads during vectorization. > Dependence analysis currently isn't clever enough to disambiguate them > thus we insert versioning-for-alias checks.  For the testcase hoisting > the load is sti

Re: [PATCH] Handle vectorization of invariant loads (PR46787)

2011-07-03 Thread Ira Rosen
Richard Guenther wrote on 30/06/2011 06:24:50 PM: > FYI, I'm testing the following which cures a fallout seen when > building SPEC2k6 with the committed patch. It's suboptimal for > j != 0 though - is there a way to get to the vectorized stmt > of the j == 0 iteration? Yes, I think we can si

Re: [PATCH] Handle vectorization of invariant loads (PR46787)

2011-06-30 Thread H.J. Lu
On Thu, Jun 30, 2011 at 8:24 AM, Richard Guenther wrote: > On Wed, 29 Jun 2011, Richard Guenther wrote: > >> >> The following patch makes us handle invariant loads during vectorization. >> Dependence analysis currently isn't clever enough to disambiguate them >> thus we insert versioning-for-alias

Re: [PATCH] Handle vectorization of invariant loads (PR46787)

2011-06-30 Thread Richard Guenther
On Wed, 29 Jun 2011, Richard Guenther wrote: > > The following patch makes us handle invariant loads during vectorization. > Dependence analysis currently isn't clever enough to disambiguate them > thus we insert versioning-for-alias checks. For the testcase hoisting > the load is still always p

Re: [PATCH] Handle vectorization of invariant loads (PR46787)

2011-06-29 Thread Ira Rosen
Richard Guenther wrote on 29/06/2011 02:19:40 PM: > > The following patch makes us handle invariant loads during vectorization. > Dependence analysis currently isn't clever enough to disambiguate them > thus we insert versioning-for-alias checks. For the testcase hoisting > the load is still a

[PATCH] Handle vectorization of invariant loads (PR46787)

2011-06-29 Thread Richard Guenther
The following patch makes us handle invariant loads during vectorization. Dependence analysis currently isn't clever enough to disambiguate them thus we insert versioning-for-alias checks. For the testcase hoisting the load is still always possible though, and for a read-after-write dependence it