Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-04-06 Thread Ira Rosen
On 6 April 2011 16:07, Hans-Peter Nilsson wrote: >> Date: Thu, 31 Mar 2011 13:39:05 +0200 >> From: Ira Rosen > >> This patch changes NEON's default vector size from 64 to 128 bits. > > I'm wondering, are there NEON-specific measurements to support > this change? > > A colleague of mine implemente

Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-04-06 Thread Hans-Peter Nilsson
> Date: Thu, 31 Mar 2011 13:39:05 +0200 > From: Ira Rosen > This patch changes NEON's default vector size from 64 to 128 bits. I'm wondering, are there NEON-specific measurements to support this change? A colleague of mine implemented support for 64- and 128-bit NEON for RAPP

Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-04-05 Thread Ira Rosen
On 6 April 2011 08:22, Ira Rosen wrote: > > Added a RejectNegative, changed "illegal" to "invalid" and prepared a > wwwdocs patch. > The testing finished successfully, except for the test in PR > target/48454 that fails with -mvectorize-with-neon-quad without the > patch as well. There is an unre

Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-04-05 Thread Ira Rosen
On 5 April 2011 15:30, Ramana Radhakrishnan wrote: > On 31/03/11 12:39, Ira Rosen wrote: >> >> Hi, >> >> This patch changes NEON's default vector size from 64 to 128 bits. >> >> The patch doesn't touch mvectorize-with-neon-quad, but removes the >> uses of TARGET_NEON_VECTORIZE_QUAD. >> Following J

Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-04-05 Thread Ramana Radhakrishnan
On 31/03/11 12:39, Ira Rosen wrote: Hi, This patch changes NEON's default vector size from 64 to 128 bits. The patch doesn't touch mvectorize-with-neon-quad, but removes the uses of TARGET_NEON_VECTORIZE_QUAD. Following Julian's suggestion I added a param preferred-vector-size for testing and d

Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-03-31 Thread Ira Rosen
On 31 March 2011 15:11, Nathan Froyd wrote: > On Thu, Mar 31, 2011 at 01:39:05PM +0200, Ira Rosen wrote: >> This patch changes NEON's default vector size from 64 to 128 bits. > > No comments about the patch itself, but this change should be noted in > changes.html. I'll do that. Thanks, Ira > >

Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-03-31 Thread Nathan Froyd
On Thu, Mar 31, 2011 at 01:39:05PM +0200, Ira Rosen wrote: > This patch changes NEON's default vector size from 64 to 128 bits. No comments about the patch itself, but this change should be noted in changes.html. -Nathan

Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-03-31 Thread Ira Rosen
On 31 March 2011 14:28, Joseph S. Myers wrote: > On Thu, 31 Mar 2011, Ira Rosen wrote: > >> +Illegal values are ignored.  The default is 128. > > See the GNU Coding Standards > : > >   Please do not use the term "illegal" to refer to er

Re: [patch, ARM] Make 128 bits the default vector size for NEON

2011-03-31 Thread Joseph S. Myers
On Thu, 31 Mar 2011, Ira Rosen wrote: > +Illegal values are ignored. The default is 128. See the GNU Coding Standards : Please do not use the term "illegal" to refer to erroneous input to a computer program. Please use "invali