Fix a minor, but bitstream-affecting bug
[opus.git] / libcelt / vq.c
index 7407151..4876b59 100644 (file)
@@ -1,5 +1,6 @@
-/* (C) 2007 Jean-Marc Valin, CSIRO
-*/
+/* Copyright (c) 2007-2008 CSIRO
+   Copyright (c) 2007-2009 Xiph.Org Foundation
+   Written by Jean-Marc Valin */
 /*
    Redistribution and use in source and binary forms, with or without
    modification, are permitted provided that the following conditions
 #include "vq.h"
 #include "arch.h"
 #include "os_support.h"
+#include "rate.h"
+
+#ifndef M_PI
+#define M_PI 3.141592653
+#endif
+
+static celt_uint32 lcg_rand(celt_uint32 seed)
+{
+   return 1664525 * seed + 1013904223;
+}
 
-/** Takes the pitch vector and the decoded residual vector (non-compressed), 
-   applies the compression in the pitch direction, computes the gain that will
-   give ||p+g*y||=1 and mixes the residual with the pitch. */
-static void mix_pitch_and_residual(int *iy, celt_norm_t *X, int N, int K, const celt_norm_t *P)
+static void exp_rotation1(celt_norm *X, int len, int stride, celt_word16 c, celt_word16 s)
 {
    int i;
-   celt_word32_t Ryp, Ryy, Rpp;
-   celt_word32_t g;
-   VARDECL(celt_norm_t, y);
-#ifdef FIXED_POINT
-   int yshift;
-#endif
-   SAVE_STACK;
-#ifdef FIXED_POINT
-   yshift = 14-EC_ILOG(K);
-#endif
-   ALLOC(y, N, celt_norm_t);
+   celt_norm *Xptr;
+   Xptr = X;
+   for (i=0;i<len-stride;i++)
+   {
+      celt_norm x1, x2;
+      x1 = Xptr[0];
+      x2 = Xptr[stride];
+      Xptr[stride] = EXTRACT16(SHR32(MULT16_16(c,x2) + MULT16_16(s,x1), 15));
+      *Xptr++      = EXTRACT16(SHR32(MULT16_16(c,x1) - MULT16_16(s,x2), 15));
+   }
+   Xptr = &X[len-2*stride-1];
+   for (i=len-2*stride-1;i>=0;i--)
+   {
+      celt_norm x1, x2;
+      x1 = Xptr[0];
+      x2 = Xptr[stride];
+      Xptr[stride] = EXTRACT16(SHR32(MULT16_16(c,x2) + MULT16_16(s,x1), 15));
+      *Xptr--      = EXTRACT16(SHR32(MULT16_16(c,x1) - MULT16_16(s,x2), 15));
+   }
+}
 
-   /*for (i=0;i<N;i++)
-   printf ("%d ", iy[i]);*/
-   Rpp = 0;
-   for (i=0;i<N;i++)
-      Rpp = MAC16_16(Rpp,P[i],P[i]);
+static void exp_rotation(celt_norm *X, int len, int dir, int stride, int K, int spread)
+{
+   int i;
+   celt_word16 c, s;
+   celt_word16 gain, theta;
+   int stride2=0;
+   int factor;
+   /*int i;
+   if (len>=30)
+   {
+      for (i=0;i<len;i++)
+         X[i] = 0;
+      X[14] = 1;
+      K=5;
+   }*/
+   if (2*K>=len || spread==0)
+      return;
+   if (spread==1)
+      factor=10;
+   else if (spread==2)
+      factor=5;
+   else
+      factor=15;
 
-   Ryp = 0;
-   for (i=0;i<N;i++)
-      Ryp = MAC16_16(Ryp,SHL16(iy[i],yshift),P[i]);
+   gain = celt_div((celt_word32)MULT16_16(Q15_ONE,len),(celt_word32)(len+factor*K));
+   /* FIXME: Make that HALF16 instead of HALF32 */
+   theta = HALF32(MULT16_16_Q15(gain,gain));
 
-   /* Remove part of the pitch component to compute the real residual from
-   the encoded (int) one */
-   for (i=0;i<N;i++)
-      y[i] = SHL16(iy[i],yshift);
+   c = celt_cos_norm(EXTEND32(theta));
+   s = celt_cos_norm(EXTEND32(SUB16(Q15ONE,theta))); /*  sin(theta) */
 
-   /* Recompute after the projection (I think it's right) */
-   Ryp = 0;
-   for (i=0;i<N;i++)
-      Ryp = MAC16_16(Ryp,y[i],P[i]);
+   if (len>=8*stride)
+   {
+      stride2 = 1;
+      /* This is just a simple way of computing sqrt(len/stride) with rounding.
+         It's basically incrementing long as (stride2+0.5)^2 < len/stride.
+         I _think_ it is bit-exact */
+      while ((stride2*stride2+stride2)*stride + (stride>>2) < len)
+         stride2++;
+   }
+   len /= stride;
+   for (i=0;i<stride;i++)
+   {
+      if (dir < 0)
+      {
+         if (stride2)
+            exp_rotation1(X+i*len, len, stride2, s, c);
+         exp_rotation1(X+i*len, len, 1, c, s);
+      } else {
+         exp_rotation1(X+i*len, len, 1, c, -s);
+         if (stride2)
+            exp_rotation1(X+i*len, len, stride2, s, -c);
+      }
+   }
+   /*if (len>=30)
+   {
+      for (i=0;i<len;i++)
+         printf ("%f ", X[i]);
+      printf ("\n");
+      exit(0);
+   }*/
+}
 
-   Ryy = 0;
-   for (i=0;i<N;i++)
-      Ryy = MAC16_16(Ryy, y[i],y[i]);
+/** Takes the pitch vector and the decoded residual vector, computes the gain
+    that will give ||p+g*y||=1 and mixes the residual with the pitch. */
+static void normalise_residual(int * restrict iy, celt_norm * restrict X, int N, int K, celt_word32 Ryy)
+{
+   int i;
+#ifdef FIXED_POINT
+   int k;
+#endif
+   celt_word32 t;
+   celt_word16 g;
 
-   /* g = (sqrt(Ryp^2 + Ryy - Rpp*Ryy)-Ryp)/Ryy */
-   g = MULT16_32_Q15(
-            celt_sqrt(MULT16_16(ROUND16(Ryp,14),ROUND16(Ryp,14)) + Ryy -
-                      MULT16_16(ROUND16(Ryy,14),ROUND16(Rpp,14)))
-            - ROUND16(Ryp,14),
-       celt_rcp(SHR32(Ryy,9)));
+#ifdef FIXED_POINT
+   k = celt_ilog2(Ryy)>>1;
+#endif
+   t = VSHR32(Ryy, (k-7)<<1);
+   g = celt_rsqrt_norm(t);
 
-   for (i=0;i<N;i++)
-      X[i] = P[i] + ROUND16(MULT16_16(y[i], g),11);
-   RESTORE_STACK;
+   i=0;
+   do
+      X[i] = EXTRACT16(PSHR32(MULT16_16(g, iy[i]), k+1));
+   while (++i < N);
 }
 
-
-void alg_quant(celt_norm_t *X, celt_mask_t *W, int N, int K, const celt_norm_t *P, ec_enc *enc)
+void alg_quant(celt_norm *X, int N, int K, int spread, int B, celt_norm *lowband, int resynth, ec_enc *enc, celt_int32 *seed)
 {
-   VARDECL(celt_norm_t, y);
+   VARDECL(celt_norm, y);
    VARDECL(int, iy);
-   VARDECL(int, signx);
-   VARDECL(celt_word32_t, scores);
-   int i, j, is;
-   celt_word16_t s;
+   VARDECL(celt_word16, signx);
+   int j, is;
+   celt_word16 s;
    int pulsesLeft;
-   celt_word32_t sum;
-   celt_word32_t xy, yy, yp;
-   celt_word16_t Rpp;
+   celt_word32 sum;
+   celt_word32 xy, yy;
+   int N_1; /* Inverse of N, in Q14 format (even for float) */
 #ifdef FIXED_POINT
    int yshift;
 #endif
    SAVE_STACK;
 
+   /* When there's no pulse, fill with noise or folded spectrum */
+   if (K==0)
+   {
+      if (lowband != NULL && resynth)
+      {
+         for (j=0;j<N;j++)
+            X[j] = lowband[j];
+      } else {
+         /* This is important for encoding the side in stereo mode */
+         for (j=0;j<N;j++)
+         {
+            *seed = lcg_rand(*seed);
+            X[j] = (int)(*seed)>>20;
+         }
+      }
+      renormalise_vector(X, Q15ONE, N, 1);
+      return;
+   }
+   K = get_pulses(K);
 #ifdef FIXED_POINT
-   yshift = 14-EC_ILOG(K);
+   yshift = 13-celt_ilog2(K);
 #endif
 
-   ALLOC(y, N, celt_norm_t);
+   ALLOC(y, N, celt_norm);
    ALLOC(iy, N, int);
-   ALLOC(signx, N, int);
-   ALLOC(scores, N, celt_word32_t);
+   ALLOC(signx, N, celt_word16);
+   N_1 = 512/N;
+   
+   exp_rotation(X, N, 1, B, K, spread);
 
-   for (j=0;j<N;j++)
-   {
+   /* Get rid of the sign */
+   sum = 0;
+   j=0; do {
       if (X[j]>0)
          signx[j]=1;
-      else
+      else {
          signx[j]=-1;
-   }
-   
-   sum = 0;
-   for (j=0;j<N;j++)
-   {
-      sum = MAC16_16(sum, P[j],P[j]);
-   }
-   Rpp = ROUND16(sum, NORM_SHIFT);
-
-   celt_assert2(Rpp<=NORM_SCALING, "Rpp should never have a norm greater than unity");
+         X[j]=-X[j];
+      }
+      iy[j] = 0;
+      y[j] = 0;
+   } while (++j<N);
 
-   for (i=0;i<N;i++)
-      y[i] = 0;
-   for (i=0;i<N;i++)
-      iy[i] = 0;
-   xy = yy = yp = 0;
+   xy = yy = 0;
 
    pulsesLeft = K;
+
+   /* Do a pre-search by projecting on the pyramid */
+   if (K > (N>>1))
+   {
+      celt_word16 rcp;
+      j=0; do {
+         sum += X[j];
+      }  while (++j<N);
+
+      /* If X is too small, just replace it with a pulse at 0 */
+#ifdef FIXED_POINT
+      if (sum <= K)
+#else
+      if (sum <= EPSILON)
+#endif
+      {
+         X[0] = QCONST16(1.f,14);
+         j=1; do
+            X[j]=0;
+         while (++j<N);
+         sum = QCONST16(1.f,14);
+      }
+      /* Do we have sufficient accuracy here? */
+      rcp = EXTRACT16(MULT16_32_Q16(K-1, celt_rcp(sum)));
+      j=0; do {
+#ifdef FIXED_POINT
+         /* It's really important to round *towards zero* here */
+         iy[j] = MULT16_16_Q15(X[j],rcp);
+#else
+         iy[j] = (int)floor(rcp*X[j]);
+#endif
+         y[j] = SHL16(iy[j],yshift);
+         yy = MAC16_16(yy, y[j],y[j]);
+         xy = MAC16_16(xy, X[j],y[j]);
+         y[j] *= 2;
+         pulsesLeft -= iy[j];
+      }  while (++j<N);
+   }
+   celt_assert2(pulsesLeft>=1, "Allocated too many pulses in the quick pass");
+
    while (pulsesLeft > 0)
    {
       int pulsesAtOnce=1;
-      int sign;
-      celt_word32_t Rxy, Ryy, Ryp;
-      celt_word32_t g;
-      
+      int best_id;
+      celt_word16 magnitude;
+      celt_word32 best_num = -VERY_LARGE16;
+      celt_word16 best_den = 0;
+#ifdef FIXED_POINT
+      int rshift;
+#endif
       /* Decide on how many pulses to find at once */
-      pulsesAtOnce = pulsesLeft/N;
+      pulsesAtOnce = (pulsesLeft*N_1)>>9; /* pulsesLeft/N */
       if (pulsesAtOnce<1)
          pulsesAtOnce = 1;
-      /*printf ("%d %d %d/%d %d\n", Lupdate, pulsesAtOnce, pulsesLeft, K, N);*/
+#ifdef FIXED_POINT
+      rshift = yshift+1+celt_ilog2(K-pulsesLeft+pulsesAtOnce);
+#endif
+      magnitude = SHL16(pulsesAtOnce, yshift);
 
+      best_id = 0;
+      /* The squared magnitude term gets added anyway, so we might as well 
+         add it outside the loop */
+      yy = MAC16_16(yy, magnitude,magnitude);
       /* Choose between fast and accurate strategy depending on where we are in the search */
-      if (pulsesLeft>1)
-      {
-         for (j=0;j<N;j++)
-         {
-            /* Select sign based on X[j] alone */
-            sign = signx[j];
-            s = SHL16(sign*pulsesAtOnce, yshift);
-            /* Temporary sums of the new pulse(s) */
-            Rxy = xy + MULT16_16(s,X[j]);
-            Ryy = yy + 2*MULT16_16(s,y[j]) + MULT16_16(s,s);
-            Ryp = yp + MULT16_16(s, P[j]);
-            scores[j] = MULT32_32_Q31(MULT16_16(ROUND16(Rxy,14),ABS16(ROUND16(Rxy,14))), celt_rcp(SHR32(Ryy,12)));
-         }
-      } else {
-         for (j=0;j<N;j++)
+         /* This should ensure that anything we can process will have a better score */
+      j=0;
+      do {
+         celt_word16 Rxy, Ryy;
+         /* Select sign based on X[j] alone */
+         s = magnitude;
+         /* Temporary sums of the new pulse(s) */
+         Rxy = EXTRACT16(SHR32(MAC16_16(xy, s,X[j]),rshift));
+         /* We're multiplying y[j] by two so we don't have to do it here */
+         Ryy = EXTRACT16(SHR32(MAC16_16(yy, s,y[j]),rshift));
+            
+         /* Approximate score: we maximise Rxy/sqrt(Ryy) (we're guaranteed that
+            Rxy is positive because the sign is pre-computed) */
+         Rxy = MULT16_16_Q15(Rxy,Rxy);
+         /* The idea is to check for num/den >= best_num/best_den, but that way
+            we can do it without any division */
+         /* OPT: Make sure to use conditional moves here */
+         if (MULT16_16(best_den, Rxy) > MULT16_16(Ryy, best_num))
          {
-            /* Select sign based on X[j] alone */
-            sign = signx[j];
-            s = SHL16(sign*pulsesAtOnce, yshift);
-            /* Temporary sums of the new pulse(s) */
-            Rxy = xy + MULT16_16(s,X[j]);
-            Ryy = yy + 2*MULT16_16(s,y[j]) + MULT16_16(s,s);
-            Ryp = yp + MULT16_16(s, P[j]);
-
-            /* Compute the gain such that ||p + g*y|| = 1 */
-            g = MULT16_32_Q15(
-                     celt_sqrt(MULT16_16(ROUND16(Ryp,14),ROUND16(Ryp,14)) + Ryy -
-                               MULT16_16(ROUND16(Ryy,14),Rpp))
-                     - ROUND16(Ryp,14),
-                celt_rcp(SHR32(Ryy,12)));
-            /* Knowing that gain, what's the error: (x-g*y)^2 
-               (result is negated and we discard x^2 because it's constant) */
-            /* score = 2.f*g*Rxy - 1.f*g*g*Ryy*NORM_SCALING_1;*/
-            scores[j] = 2*MULT16_32_Q14(ROUND16(Rxy,14),g)
-                    - MULT16_32_Q14(EXTRACT16(MULT16_32_Q14(ROUND16(Ryy,14),g)),g);
+            best_den = Ryy;
+            best_num = Rxy;
+            best_id = j;
          }
-      }
+      } while (++j<N);
       
-      j = find_max32(scores, N);
-      is = signx[j]*pulsesAtOnce;
+      j = best_id;
+      is = pulsesAtOnce;
       s = SHL16(is, yshift);
 
       /* Updating the sums of the new pulse(s) */
       xy = xy + MULT16_16(s,X[j]);
-      yy = yy + 2*MULT16_16(s,y[j]) + MULT16_16(s,s);
-      yp = yp + MULT16_16(s, P[j]);
+      /* We're multiplying y[j] by two so we don't have to do it here */
+      yy = yy + MULT16_16(s,y[j]);
 
       /* Only now that we've made the final choice, update y/iy */
-      y[j] += s;
+      /* Multiplying y[j] by 2 so we don't have to do it everywhere else */
+      y[j] += 2*s;
       iy[j] += is;
       pulsesLeft -= pulsesAtOnce;
    }
-   
+
+   /* Put the original sign back */
+   j=0;
+   do {
+      X[j] = MULT16_16(signx[j],X[j]);
+      if (signx[j] < 0)
+         iy[j] = -iy[j];
+   } while (++j<N);
    encode_pulses(iy, N, K, enc);
    
-   /* Recompute the gain in one pass to reduce the encoder-decoder mismatch
-   due to the recursive computation used in quantisation. */
-   mix_pitch_and_residual(iy, X, N, K, P);
+   if (resynth)
+   {
+      normalise_residual(iy, X, N, K, EXTRACT16(SHR32(yy,2*yshift)));
+      exp_rotation(X, N, -1, B, K, spread);
+   }
    RESTORE_STACK;
 }
 
 
 /** Decode pulse vector and combine the result with the pitch vector to produce
     the final normalised signal in the current band. */
-void alg_unquant(celt_norm_t *X, int N, int K, celt_norm_t *P, ec_dec *dec)
+void alg_unquant(celt_norm *X, int N, int K, int spread, int B, celt_norm *lowband, ec_dec *dec, celt_int32 *seed)
 {
+   int i;
+   celt_word32 Ryy;
    VARDECL(int, iy);
    SAVE_STACK;
-   ALLOC(iy, N, int);
-   decode_pulses(iy, N, K, dec);
-   mix_pitch_and_residual(iy, X, N, K, P);
-   RESTORE_STACK;
-}
-
-#ifdef FIXED_POINT
-static const celt_word16_t pg[11] = {32767, 24576, 21299, 19661, 19661, 19661, 18022, 18022, 16384, 16384, 16384};
-#else
-static const celt_word16_t pg[11] = {1.f, .75f, .65f, 0.6f, 0.6f, .6f, .55f, .55f, .5f, .5f, .5f};
-#endif
 
-#define MAX_INTRA 32
-#define LOG_MAX_INTRA 5
-      
-void intra_prediction(celt_norm_t *x, celt_mask_t *W, int N, int K, celt_norm_t *Y, celt_norm_t *P, int B, int N0, ec_enc *enc)
-{
-   int i,j;
-   int best=0;
-   celt_word32_t best_score=0;
-   celt_word16_t s = 1;
-   int sign;
-   celt_word32_t E;
-   celt_word16_t pred_gain;
-   int max_pos = N0-N/B;
-   if (max_pos > MAX_INTRA)
-      max_pos = MAX_INTRA;
-
-   for (i=0;i<max_pos*B;i+=B)
+   if (K==0)
    {
-      celt_word32_t xy=0, yy=0;
-      celt_word32_t score;
-      /* If this doesn't generate a double-MAC on supported architectures, 
-         complain to your compilor vendor */
-      for (j=0;j<N;j++)
+      if (lowband != NULL)
       {
-         xy = MAC16_16(xy, x[j], Y[i+N-j-1]);
-         yy = MAC16_16(yy, Y[i+N-j-1], Y[i+N-j-1]);
-      }
-      /* If you're really desperate for speed, just use xy as the score */
-      score = celt_div(MULT16_16(ROUND16(xy,14),ROUND16(xy,14)), ROUND16(yy,14));
-      if (score > best_score)
-      {
-         best_score = score;
-         best = i;
-         /* Store xy as the sign. We'll normalise it to +/- 1 later. */
-         s = ROUND16(xy,14);
+         for (i=0;i<N;i++)
+            X[i] = lowband[i];
+      } else {
+         /* This is important for encoding the side in stereo mode */
+         for (i=0;i<N;i++)
+         {
+            *seed = lcg_rand(*seed);
+            X[i] = (int)(*seed)>>20;
+         }
       }
+      renormalise_vector(X, Q15ONE, N, 1);
+      return;
    }
-   if (s<0)
-   {
-      s = -1;
-      sign = 1;
-   } else {
-      s = 1;
-      sign = 0;
-   }
-   /*printf ("%d %d ", sign, best);*/
-   ec_enc_bits(enc,sign,1);
-   if (max_pos == MAX_INTRA)
-      ec_enc_bits(enc,best/B,LOG_MAX_INTRA);
-   else
-      ec_enc_uint(enc,best/B,max_pos);
-
-   /*printf ("%d %f\n", best, best_score);*/
-   
-   if (K>10)
-      pred_gain = pg[10];
-   else
-      pred_gain = pg[K];
-   E = EPSILON;
-   for (j=0;j<N;j++)
-   {
-      P[j] = s*Y[best+N-j-1];
-      E = MAC16_16(E, P[j],P[j]);
-   }
-   /*pred_gain = pred_gain/sqrt(E);*/
-   pred_gain = MULT16_16_Q15(pred_gain,celt_rcp(SHL32(celt_sqrt(E),9)));
-   for (j=0;j<N;j++)
-      P[j] = PSHR32(MULT16_16(pred_gain, P[j]),8);
-   if (K>0)
-   {
-      for (j=0;j<N;j++)
-         x[j] -= P[j];
-   } else {
-      for (j=0;j<N;j++)
-         x[j] = P[j];
-   }
-   /*printf ("quant ");*/
-   /*for (j=0;j<N;j++) printf ("%f ", P[j]);*/
-
+   K = get_pulses(K);
+   ALLOC(iy, N, int);
+   decode_pulses(iy, N, K, dec);
+   Ryy = 0;
+   i=0;
+   do {
+      Ryy = MAC16_16(Ryy, iy[i], iy[i]);
+   } while (++i < N);
+   normalise_residual(iy, X, N, K, Ryy);
+   exp_rotation(X, N, -1, B, K, spread);
+   RESTORE_STACK;
 }
 
-void intra_unquant(celt_norm_t *x, int N, int K, celt_norm_t *Y, celt_norm_t *P, int B, int N0, ec_dec *dec)
+celt_word16 renormalise_vector(celt_norm *X, celt_word16 value, int N, int stride)
 {
-   int j;
-   int sign;
-   celt_word16_t s;
-   int best;
-   celt_word32_t E;
-   celt_word16_t pred_gain;
-   int max_pos = N0-N/B;
-   if (max_pos > MAX_INTRA)
-      max_pos = MAX_INTRA;
-   
-   sign = ec_dec_bits(dec, 1);
-   if (sign == 0)
-      s = 1;
-   else
-      s = -1;
-   
-   if (max_pos == MAX_INTRA)
-      best = B*ec_dec_bits(dec, LOG_MAX_INTRA);
-   else
-      best = B*ec_dec_uint(dec, max_pos);
-   /*printf ("%d %d ", sign, best);*/
-
-   if (K>10)
-      pred_gain = pg[10];
-   else
-      pred_gain = pg[K];
-   E = EPSILON;
-   for (j=0;j<N;j++)
-   {
-      P[j] = s*Y[best+N-j-1];
-      E = MAC16_16(E, P[j],P[j]);
-   }
-   /*pred_gain = pred_gain/sqrt(E);*/
-   pred_gain = MULT16_16_Q15(pred_gain,celt_rcp(SHL32(celt_sqrt(E),9)));
-   for (j=0;j<N;j++)
-      P[j] = PSHR32(MULT16_16(pred_gain, P[j]),8);
-   if (K==0)
+   int i;
+#ifdef FIXED_POINT
+   int k;
+#endif
+   celt_word32 E = EPSILON;
+   celt_word16 g;
+   celt_word32 t;
+   celt_norm *xptr = X;
+   for (i=0;i<N;i++)
    {
-      for (j=0;j<N;j++)
-         x[j] = P[j];
+      E = MAC16_16(E, *xptr, *xptr);
+      xptr += stride;
    }
-}
+#ifdef FIXED_POINT
+   k = celt_ilog2(E)>>1;
+#endif
+   t = VSHR32(E, (k-7)<<1);
+   g = MULT16_16_Q15(value, celt_rsqrt_norm(t));
 
-void intra_fold(celt_norm_t *x, int N, celt_norm_t *Y, celt_norm_t *P, int B, int N0, int Nmax)
-{
-   int i, j;
-   celt_word32_t E;
-   celt_word16_t g;
-   
-   E = EPSILON;
-   if (N0 >= (Nmax>>1))
+   xptr = X;
+   for (i=0;i<N;i++)
    {
-      for (i=0;i<B;i++)
-      {
-         for (j=0;j<N/B;j++)
-         {
-            P[j*B+i] = Y[(Nmax-N0-j-1)*B+i];
-            E += P[j*B+i]*P[j*B+i];
-         }
-      }
-   } else {
-      for (j=0;j<N;j++)
-      {
-         P[j] = Y[j];
-         E = MAC16_16(E, P[j],P[j]);
-      }
+      *xptr = EXTRACT16(PSHR32(MULT16_16(g, *xptr), k+1));
+      xptr += stride;
    }
-   g = celt_rcp(SHL32(celt_sqrt(E),9));
-   for (j=0;j<N;j++)
-      P[j] = PSHR32(MULT16_16(g, P[j]),8);
-   for (j=0;j<N;j++)
-      x[j] = P[j];
+   return celt_sqrt(E);
 }