paddyhoran commented on pull request #7037: URL: https://github.com/apache/arrow/pull/7037#issuecomment-624828880
So, I'm not sure now about removing SIMD altogether. It took me some time to add all the SIMD features but I don't pretend to be an expert on it. I'd hate to remove all the SIMD code only to realize that someone smarter than I could have improved it (like @yordan-pavlov). I think we shouldn't use it by default because it requires nightly, #7057 changes it to be opt-in. I would love to remove some of the code if it's not required though. @yordan-pavlov I think the best way to move this forward is if you can contribute to the benchmarks for the main arrow repo. Then we can look at removing the SIMD code piece by piece to ensure we don't have performance regressions. Thoughts? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
