Skip to content

Commit 4cfd6f1

Browse files
authored
Merge pull request #3678 from martin-frbg/issue3677
Eliminate uses of CREAL on left-hand side of assignments
2 parents 686e6d7 + e12d474 commit 4cfd6f1

File tree

1 file changed

+6
-9
lines changed

1 file changed

+6
-9
lines changed

kernel/arm64/zdot_thunderx2t99.c

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -198,8 +198,8 @@ extern int blas_level1_thread_with_return_value(int mode, BLASLONG m, BLASLONG n
198198
static void zdot_compute(BLASLONG n, FLOAT *x, BLASLONG inc_x, FLOAT *y, BLASLONG inc_y, OPENBLAS_COMPLEX_FLOAT *result)
199199
{
200200
FLOAT dotr = 0.0, doti = 0.0;
201-
CREAL(*result) = 0.0;
202-
CIMAG(*result) = 0.0;
201+
OPENBLAS_COMPLEX_FLOAT cf = OPENBLAS_MAKE_COMPLEX_FLOAT(0.0, 0.0);
202+
*result = cf;
203203

204204
if ( n < 0 ) return;
205205

@@ -290,8 +290,8 @@ static void zdot_compute(BLASLONG n, FLOAT *x, BLASLONG inc_x, FLOAT *y, BLASLON
290290
"d0", "d1", "d2", "d3", "d4", "d5", "d6", "d7"
291291
);
292292

293-
CREAL(*result) = dotr;
294-
CIMAG(*result) = doti;
293+
cf=OPENBLAS_MAKE_COMPLEX_FLOAT(dotr, doti);
294+
*result = cf;
295295
return;
296296
}
297297

@@ -312,9 +312,7 @@ OPENBLAS_COMPLEX_FLOAT CNAME(BLASLONG n, FLOAT *x, BLASLONG inc_x, FLOAT *y, BLA
312312
int nthreads;
313313
FLOAT dummy_alpha;
314314
#endif
315-
OPENBLAS_COMPLEX_FLOAT zdot;
316-
CREAL(zdot) = 0.0;
317-
CIMAG(zdot) = 0.0;
315+
OPENBLAS_COMPLEX_FLOAT zdot = OPENBLAS_MAKE_COMPLEX_FLOAT(0.0,0.0);
318316

319317
#if defined(SMP)
320318
if (inc_x == 0 || inc_y == 0 || n <= 10000)
@@ -341,8 +339,7 @@ OPENBLAS_COMPLEX_FLOAT CNAME(BLASLONG n, FLOAT *x, BLASLONG inc_x, FLOAT *y, BLA
341339

342340
ptr = (OPENBLAS_COMPLEX_FLOAT *)result;
343341
for (i = 0; i < nthreads; i++) {
344-
CREAL(zdot) = CREAL(zdot) + CREAL(*ptr);
345-
CIMAG(zdot) = CIMAG(zdot) + CIMAG(*ptr);
342+
zdot = OPENBLAS_MAKE_COMPLEX_FLOAT (CREAL(zdot) + CREAL(*ptr), CIMAG(zdot) + CIMAG(*ptr));
346343
ptr = (void *)(((char *)ptr) + sizeof(double) * 2);
347344
}
348345
}

0 commit comments

Comments
 (0)