nevi-me commented on pull request #6898: URL: https://github.com/apache/arrow/pull/6898#issuecomment-619797480
> > I am using this code in various places, but if it is against the spec or not useful for the improvements we can drop it off. > > As @pitrou says: > > > It's a recommendation, not an obligation. If you decide that specific architectures deserve a specific alignment value, then IMHO it's not a problem. > > I'm all for the changes if they improve performance but this is at the limit of my understanding. @andygrove @nevi-me thoughts? I don't know much about alignment, but I'm happy to claw back performance (esp after we remove packed_simd). I'll rebase this against the PR to drop packed_simd, then check the performance difference. ---------------------------------------------------------------- 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: us...@infra.apache.org