diff options
| author | Frank Barchard <fbarchard@google.com> | 2022-08-24 15:03:32 -0700 |
|---|---|---|
| committer | XNNPACK Team <xnnpack-github-robot@google.com> | 2022-08-24 15:04:31 -0700 |
| commit | cceec276e00405417c7caadfe8e3ef7f8f0af8b3 (patch) | |
| tree | 458829418897901681775d52836b9d9c9ed17e37 | |
| parent | b1aad88bb522c36dce8a9ffe1f7f0d2b2743088b (diff) | |
| download | XNNPACK-cceec276e00405417c7caadfe8e3ef7f8f0af8b3.tar.gz | |
Fix bfly4m1 tester - allocate 4 values for m1
- Was allocating and testing 1 value but issue was masked by having padding for NEON.
- Remove padding if input and twiddle. Not needed. Improves confidence in test.
PiperOrigin-RevId: 469833223
| -rw-r--r-- | test/bfly4-microkernel-tester.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/test/bfly4-microkernel-tester.h b/test/bfly4-microkernel-tester.h index ae3182b4a..f517805ed 100644 --- a/test/bfly4-microkernel-tester.h +++ b/test/bfly4-microkernel-tester.h @@ -215,11 +215,11 @@ class BFly4MicrokernelTester { std::random_device random_device; auto rng = std::mt19937(random_device()); auto i16rng = std::bind(std::uniform_int_distribution<int16_t>(), std::ref(rng)); - const size_t fft_size = samples() == 1 ? 1 : (samples() * stride()) * 4; // 4 for bfly4. + const size_t fft_size = (samples() == 1 ? 1 : (samples() * stride())) * 4; // 4 for bfly4. // 256 complex numbers = fft_size * 2 = 512 - std::vector<int16_t> y(fft_size * 2 + XNN_EXTRA_BYTES / sizeof(int16_t)); - std::vector<int16_t> y_ref(fft_size * 2 + XNN_EXTRA_BYTES / sizeof(int16_t)); + std::vector<int16_t> y(fft_size * 2); + std::vector<int16_t> y_ref(fft_size * 2); for (size_t iteration = 0; iteration < iterations(); iteration++) { std::generate(y.begin(), y.end(), std::ref(i16rng)); |