Skip to content

Ne10: Fix uninitialized variable in FFT functions #296

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
christophe0606 merged 1 commit into from
Apr 1, 2026
Merged

Ne10: Fix uninitialized variable in FFT functions #296

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions Ne10/NE10_fft_float32.neonintrinsic.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1214,7 +1214,7 @@ void arm_ne10_mixed_radix_fft_forward_float32_neon (
ne10_int32_t fstride = S->factors[1];
ne10_int32_t mstride = S->factors[3];
ne10_int32_t first_radix = S->factors[2];
ne10_int32_t step, f_count;
ne10_int32_t step = 0, f_count;
const ne10_fft_cpx_float32_t *src = input;
ne10_fft_cpx_float32_t *in;
ne10_fft_cpx_float32_t *dst = out;
Expand Down Expand Up @@ -1313,7 +1313,7 @@ void arm_ne10_mixed_radix_fft_backward_float32_neon (
ne10_int32_t mstride = S->factors[3];
ne10_int32_t first_radix = S->factors[2];
ne10_int32_t nfft = fstride * first_radix;
ne10_int32_t step, f_count;
ne10_int32_t step = 0, f_count;
const ne10_fft_cpx_float32_t *src = input;
ne10_fft_cpx_float32_t *in;
ne10_fft_cpx_float32_t *dst = out;
Expand Down
4 changes: 2 additions & 2 deletions Ne10/NE10_fft_int16.neonintrinsic.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1026,7 +1026,7 @@ static void arm_ne10_mixed_radix_fft_forward_int16_##scaled##_neon (ne10_fft_cpx
ne10_fft_cpx_int16_t *Fout1; \
ne10_fft_cpx_int16_t *Fout_ls = Fout; \
ne10_fft_cpx_int16_t *Ftmp; \
const ne10_fft_cpx_int16_t *tw; \
const ne10_fft_cpx_int16_t *tw = twiddles; \
const ne10_fft_cpx_int16_t *tw1; \
\
/* init fstride, mstride, N */ \
Expand Down Expand Up @@ -1111,7 +1111,7 @@ static void arm_ne10_mixed_radix_fft_backward_int16_##scaled##_neon (ne10_fft_cp
ne10_fft_cpx_int16_t *Fout1; \
ne10_fft_cpx_int16_t *Fout_ls = Fout; \
ne10_fft_cpx_int16_t *Ftmp; \
const ne10_fft_cpx_int16_t *tw;\
const ne10_fft_cpx_int16_t *tw = twiddles;\
const ne10_fft_cpx_int16_t *tw1; \
\
/* init fstride, mstride, N */ \
Expand Down
4 changes: 2 additions & 2 deletions Ne10/NE10_fft_int32.neonintrinsic.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1267,7 +1267,7 @@ static void arm_ne10_mixed_radix_fft_forward_int32_##scaled##_neon (ne10_fft_cpx
ne10_fft_cpx_int32_t *Fout1; \
ne10_fft_cpx_int32_t *Fout_ls = Fout; \
ne10_fft_cpx_int32_t *Ftmp; \
const ne10_fft_cpx_int32_t *tw;\
const ne10_fft_cpx_int32_t *tw = twiddles;\
const ne10_fft_cpx_int32_t *tw1; \
\
/* init fstride, mstride, N */ \
Expand Down Expand Up @@ -1353,7 +1353,7 @@ static void arm_ne10_mixed_radix_fft_backward_int32_##scaled##_neon (ne10_fft_cp
ne10_fft_cpx_int32_t *Fout1; \
ne10_fft_cpx_int32_t *Fout_ls = Fout; \
ne10_fft_cpx_int32_t *Ftmp; \
const ne10_fft_cpx_int32_t *tw; \
const ne10_fft_cpx_int32_t *tw = twiddles; \
const ne10_fft_cpx_int32_t *tw1; \
\
/* init fstride, mstride, N */ \
Expand Down
2 changes: 2 additions & 0 deletions Source/MatrixFunctions/arm_mat_mult_f64.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,7 @@ ARM_DSP_ATTRIBUTE arm_status arm_mat_mult_f64(
acc5 = vdupq_n_f64(0.0);
acc6 = vdupq_n_f64(0.0);
acc7 = vdupq_n_f64(0.0);
temp = vdupq_n_f64(0.0);

/* Compute 2 MACs simultaneously. */
colCnt = numColsA >> 1U;
Expand Down Expand Up @@ -307,6 +308,7 @@ ARM_DSP_ATTRIBUTE arm_status arm_mat_mult_f64(
pIn1 = pInA;

acc0 = vdupq_n_f64(0.0);
temp = vdupq_n_f64(0.0);

/* Compute 4 MACs simultaneously. */
colCnt = numColsA >> 1U;
Expand Down
Loading