felipecrv commented on PR #35565: URL: https://github.com/apache/arrow/pull/35565#issuecomment-1554737746
I feel like the discussion went a little bit too far on the merits of alignment and where it should be enforced, when the change @westonpace is proposing here (relaxing the very aggressive 64-BYTE alignment) should not be controversial at all: it doesn't even violate the theoretical UB concerns from compilers. I'm going to review the code and we should move the general alignment discussions to the issue instead of having it on this PR. -- 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]
