Re: [Patch ARM] Fix off by one error in neon_evpc_vrev.

2012-05-29 Thread Ramana Radhakrishnan
On 29 May 2012 18:30, Richard Henderson wrote: > On 05/26/2012 01:27 AM, Ramana Radhakrishnan wrote: >> >> -  for (i = 0; i<  nelt; i += diff) >> +  for (i = 0; i<  nelt ; i += (diff + 1)) >>      for (j = 0; j<= diff; j += 1) >> -      if (d->perm[i + j] != i + diff - j) >> -       return false;

Re: [Patch ARM] Fix off by one error in neon_evpc_vrev.

2012-05-29 Thread Richard Henderson
On 05/26/2012 01:27 AM, Ramana Radhakrishnan wrote: - for (i = 0; i< nelt; i += diff) + for (i = 0; i< nelt ; i += (diff + 1)) for (j = 0; j<= diff; j += 1) - if (d->perm[i + j] != i + diff - j) - return false; + { + /* This is guaranteed to be true as the value of