Skip to content

Commit e039f2a

Browse files
committed
fix: consistently use constants instead of hardcoded numbers in unrolled BLAS loops
--- type: pre_commit_static_analysis_report description: Results of running static analysis checks when committing changes. report: - task: lint_filenames status: passed - task: lint_editorconfig status: passed - task: lint_markdown status: na - task: lint_package_json status: na - task: lint_repl_help status: na - task: lint_javascript_src status: na - task: lint_javascript_cli status: na - task: lint_javascript_examples status: na - task: lint_javascript_tests status: na - task: lint_javascript_benchmarks status: na - task: lint_python status: na - task: lint_r status: na - task: lint_c_src status: passed - task: lint_c_examples status: na - task: lint_c_benchmarks status: na - task: lint_c_tests_fixtures status: na - task: lint_shell status: na - task: lint_typescript_declarations status: na - task: lint_typescript_tests status: na - task: lint_license_headers status: passed ---
1 parent 7276cf7 commit e039f2a

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

lib/node_modules/@stdlib/blas/base/dcopy/src/dcopy_ndarray.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,11 +55,11 @@ void API_SUFFIX(c_dcopy_ndarray)( const CBLAS_INT N, const double *X, const CBLA
5555
ix += strideX;
5656
iy += strideY;
5757
}
58-
if ( N < 7 ) {
58+
if ( N < M ) {
5959
return;
6060
}
6161
}
62-
for ( i = m; i < N; i += 7 ) {
62+
for ( i = m; i < N; i += M ) {
6363
Y[ iy ] = X[ ix ];
6464
Y[ iy+1 ] = X[ ix+1 ];
6565
Y[ iy+2 ] = X[ ix+2 ];

lib/node_modules/@stdlib/blas/base/sasum/src/sasum_ndarray.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ float API_SUFFIX(c_sasum_ndarray)( const CBLAS_INT N, const float *X, const CBLA
5454
ix += stride;
5555
}
5656
}
57-
if ( N < 6 ) {
57+
if ( N < M ) {
5858
return sum;
5959
}
6060
for ( i = m; i < N; i += M ) {

lib/node_modules/@stdlib/blas/base/scopy/src/scopy_ndarray.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,11 +55,11 @@ void API_SUFFIX(c_scopy_ndarray)( const CBLAS_INT N, const float *X, const CBLAS
5555
ix += strideX;
5656
iy += strideY;
5757
}
58-
if ( N < 7 ) {
58+
if ( N < M ) {
5959
return;
6060
}
6161
}
62-
for ( i = m; i < N; i += 7 ) {
62+
for ( i = m; i < N; i += M ) {
6363
Y[ iy ] = X[ ix ];
6464
Y[ iy+1 ] = X[ ix+1 ];
6565
Y[ iy+2 ] = X[ ix+2 ];

0 commit comments

Comments
 (0)