Skip to content

Commit 6157d09

Browse files
authored
Merge pull request #1358 from martin-frbg/unused_vars
Clean up spurious unused variables in the kernels
2 parents 8e75f7d + 3fea849 commit 6157d09

12 files changed

+16
-13
lines changed

kernel/generic/zimatcopy_cnc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
3535
int CNAME(BLASLONG rows, BLASLONG cols, FLOAT alpha_r, FLOAT alpha_i, FLOAT *a, BLASLONG lda)
3636
{
3737
BLASLONG i,j,ia;
38-
FLOAT *aptr,*bptr;
38+
FLOAT *aptr;
3939
FLOAT a0, a1;
4040

4141
if ( rows <= 0 ) return(0);

kernel/generic/zimatcopy_rn.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
3434
int CNAME(BLASLONG rows, BLASLONG cols, FLOAT alpha_r, FLOAT alpha_i, FLOAT *a, BLASLONG lda)
3535
{
3636
BLASLONG i,j,ia;
37-
FLOAT *aptr,*bptr;
37+
FLOAT *aptr;
3838
FLOAT a0, a1;
3939

4040
if ( rows <= 0 ) return(0);

kernel/generic/zimatcopy_rnc.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
3434
int CNAME(BLASLONG rows, BLASLONG cols, FLOAT alpha_r, FLOAT alpha_i, FLOAT *a, BLASLONG lda)
3535
{
3636
BLASLONG i,j,ia;
37-
FLOAT *aptr,*bptr;
37+
FLOAT *aptr;
3838
FLOAT a0, a1;
3939

4040
if ( rows <= 0 ) return(0);

kernel/setparam-ref.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -684,6 +684,9 @@ static void init_parameter(void) {
684684

685685
int l2 = get_l2_size();
686686

687+
(void) l2; /* dirty trick to suppress unused variable warning for targets */
688+
/* where the GEMM unrolling parameters do not depend on l2 */
689+
687690
TABLE_NAME.sgemm_q = SGEMM_DEFAULT_Q;
688691
TABLE_NAME.dgemm_q = DGEMM_DEFAULT_Q;
689692
TABLE_NAME.cgemm_q = CGEMM_DEFAULT_Q;

kernel/x86_64/cgemv_n_4.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,6 @@ static void add_y(BLASLONG n, FLOAT *src, FLOAT *dest, BLASLONG inc_dest,FLOAT a
216216
int CNAME(BLASLONG m, BLASLONG n, BLASLONG dummy1, FLOAT alpha_r,FLOAT alpha_i, FLOAT *a, BLASLONG lda, FLOAT *x, BLASLONG inc_x, FLOAT *y, BLASLONG inc_y, FLOAT *buffer)
217217
{
218218
BLASLONG i;
219-
BLASLONG j;
220219
FLOAT *a_ptr;
221220
FLOAT *x_ptr;
222221
FLOAT *y_ptr;

kernel/x86_64/dgemv_n_4.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,6 @@ static void add_y(BLASLONG n, FLOAT *src, FLOAT *dest, BLASLONG inc_dest)
204204
int CNAME(BLASLONG m, BLASLONG n, BLASLONG dummy1, FLOAT alpha, FLOAT *a, BLASLONG lda, FLOAT *x, BLASLONG inc_x, FLOAT *y, BLASLONG inc_y, FLOAT *buffer)
205205
{
206206
BLASLONG i;
207-
BLASLONG j;
208207
FLOAT *a_ptr;
209208
FLOAT *x_ptr;
210209
FLOAT *y_ptr;

kernel/x86_64/dsymv_U.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,6 @@ int CNAME(BLASLONG m, BLASLONG offset, FLOAT alpha, FLOAT *a, BLASLONG lda, FLOA
164164
FLOAT temp2;
165165
FLOAT *xp, *yp;
166166
FLOAT *a0,*a1,*a2,*a3;
167-
FLOAT at0,at1,at2,at3;
168167
FLOAT tmp1[4];
169168
FLOAT tmp2[4];
170169

kernel/x86_64/dtrmm_kernel_4x8_haswell.c

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -167,24 +167,28 @@ int CNAME(BLASLONG bm,BLASLONG bn,BLASLONG bk,FLOAT alpha,FLOAT* ba,FLOAT* bb,FL
167167

168168
FLOAT res4_0;
169169
FLOAT res4_1;
170+
/*
170171
FLOAT res4_2;
171172
FLOAT res4_3;
172-
173+
*/
173174
FLOAT res5_0;
174175
FLOAT res5_1;
176+
/*
175177
FLOAT res5_2;
176178
FLOAT res5_3;
177-
179+
*/
178180
FLOAT res6_0;
179181
FLOAT res6_1;
182+
/*
180183
FLOAT res6_2;
181184
FLOAT res6_3;
182-
185+
*/
183186
FLOAT res7_0;
184187
FLOAT res7_1;
188+
/*
185189
FLOAT res7_2;
186190
FLOAT res7_3;
187-
191+
*/
188192
FLOAT a0;
189193
FLOAT a1;
190194

kernel/x86_64/dtrsm_kernel_LN_bulldozer.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -438,7 +438,9 @@ static inline void solve(BLASLONG m, BLASLONG n, FLOAT *a, FLOAT *b, FLOAT *c, B
438438
*(cj + i) = bb;
439439
b ++;
440440

441+
/*
441442
BLASLONG i1 = i & -4 ;
443+
*/
442444
FLOAT t0,t1,t2,t3;
443445

444446
k=0;

kernel/x86_64/sgemv_n_4.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,6 @@ static void add_y(BLASLONG n, FLOAT *src, FLOAT *dest, BLASLONG inc_dest)
292292
int CNAME(BLASLONG m, BLASLONG n, BLASLONG dummy1, FLOAT alpha, FLOAT *a, BLASLONG lda, FLOAT *x, BLASLONG inc_x, FLOAT *y, BLASLONG inc_y, FLOAT *buffer)
293293
{
294294
BLASLONG i;
295-
BLASLONG j;
296295
FLOAT *a_ptr;
297296
FLOAT *x_ptr;
298297
FLOAT *y_ptr;

0 commit comments

Comments
 (0)