AntoinePrv commented on code in PR #49756: URL: https://github.com/apache/arrow/pull/49756#discussion_r3118870996
########## cpp/src/arrow/util/bpacking_simd_128.cc: ########## @@ -17,8 +17,10 @@ #if defined(ARROW_HAVE_NEON) # define UNPACK_PLATFORM unpack_neon +# define KERNEL_PLATFORM KernelNeon #elif defined(ARROW_HAVE_SSE4_2) # define UNPACK_PLATFORM unpack_sse4_2 +# define KERNEL_PLATFORM KernelSse42 #endif #if defined(UNPACK_PLATFORM) Review Comment: Not in this file I believe. I thing there are Arrow builds where SIMD is deactivated and this file is not excluded from the build tree (as opposed to the 256/512 versions). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
