Add WASM RELAXED-SIMD F32 RMINMAX microkernels#9940
Add WASM RELAXED-SIMD F32 RMINMAX microkernels#9940copybara-service[bot] merged 1 commit intogoogle:masterfrom
Conversation
| #endif // XNN_ENABLE_AVX512F && (XNN_ARCH_X86 || XNN_ARCH_X86_64) | ||
|
|
||
| #if XNN_ARCH_WASMSIMD || XNN_ARCH_WASMRELAXEDSIMD | ||
| #if XNN_ARCH_WASMSIMD |
There was a problem hiding this comment.
This change breaks the wasmdimd build here because when XNN_ARCH_WASMRELAXEDSIMD is enabled, this kernel is not declared:
XNNPACK/src/configs/reduce-config.c
Lines 1043 to 1044 in a4673cf
There was a problem hiding this comment.
Thanks for the comment! Do you think I should update reduce-config in this PR, or create a seperate PR to update it?
There was a problem hiding this comment.
I think it depends on what you update it to. If you want the relaxed simd config to use these new kernels, I would just update it in this PR.
There was a problem hiding this comment.
Got it. Thanks! I have uploaded the new change. Please note that it rebases with the latest change on wasm simd template from #9970.
b65fdc5 to
ff2d6b8
Compare
Add the missing WASM Relaxed simd f32 rminmax microkernels and update the reduce config to point to it.