abcbarryn commented on issue #34277: URL: https://github.com/apache/arrow/issues/34277#issuecomment-1442866464
I came up with this patch which seems to work and is cross platform. If you have hardware that supports the popcnt instruction, it will still use that, but if not it will use a standard C++ library function. Actually the function uses the hardware if present. Please feel free to test and benchmark this, I think the performance will be very close. So far I have built my patch against version 4.0.1 code because I had trouble building newer code from source. I am still working on that. I am using gcc version 7 and it seems your project has also included some code that may require gcc version 11. You can look at the link I sent earlier for some existing benchmarks. [bit_util.h.patch](https://github.com/apache/arrow/files/10821224/bit_util.h.patch) -- 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]
