AntoinePrv commented on PR #50547: URL: https://github.com/apache/arrow/pull/50547#issuecomment-5020455433
The PR looks sensible but this may not be the only limitation. From what I understood when last working on this part of the code, SSE4.2 is mandatory when building Arrow C++. I believe the reason at the time was the unconditional usage of `popcnt`, but perhaps now that we moved to `std::popcnt` this is more portable. There may be more places in the CMakeLists.txt where the default compiler options do pick a minimal SSE4.2 baseline so that would be a place to look at as well. In principle I think treating all SIMD as optional makes the code more uniform, and is a free abstraction since the dispatch will change to static when possible. @domibel I'm curious what is your use case? Do you only wish to disable SIMD, or do you need to target below SSE4.2? -- 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]
