commit 08fc70ba8de6c677dfec7556096144bbf0ad8cb5
author James Zern <jzern@google.com> Mon Mar 21 13:08:32 2022 -0700
committer James Zern <jzern@google.com> Mon Mar 21 23:42:49 2022 +0000
tree d1dc5c2caca5590022dfeedc1e9495a1cf47c39d
parent 1191d85a15b17cebf8a3bc71f805dbfe1f9bbd4a

av1_steepest_descent_sparse: rm set but unused variable

quiets warning under clang:
av1/encoder/sparse_linear_solver.c:427:34: warning: variable 'diff' set
but not used [-Wunused-but-set-variable]
  double resi2, resi2_last, dAd, diff, temp;
                                 ^

Change-Id: Icc1c7b81be2460de6a08c358944e4cf3ff5980c4

av1/encoder/sparse_linear_solver.c

diff --git a/av1/encoder/sparse_linear_solver.c b/av1/encoder/sparse_linear_solver.c
index 1600830..e47c78e 100644
--- a/av1/encoder/sparse_linear_solver.c
+++ b/av1/encoder/sparse_linear_solver.c
@@ -424,7 +424,7 @@
 int av1_steepest_descent_sparse(const SPARSE_MTX *A, const double *b, int bl,
                                 double *x) {
   double *d = NULL, *Ad = NULL, *Ax = NULL;
-  double resi2, resi2_last, dAd, diff, temp;
+  double resi2, resi2_last, dAd, temp;
 
   d = aom_calloc(bl, sizeof(*d));
   Ax = aom_calloc(bl, sizeof(*Ax));
@@ -448,11 +448,9 @@
     // get A*x_last
     av1_mtx_vect_multi_right(A, d, Ad, bl);
     dAd = resi2 * bl / av1_vect_vect_multi(d, bl, Ad);
-    diff = 0;
     for (i = 0; i < bl; i++) {
       temp = dAd * d[i];
       x[i] = x[i] + temp;
-      diff += temp * temp;
     }
     av1_mtx_vect_multi_right(A, x, Ax, bl);
     resi2_last = resi2;