> Testing for "( defined(__sun) && defined(__SVR4))" actually assumes that > Solaris runs on SPARC only which is not the case. Solaris runs on x86 > as well and x86 doesn't have these alignment issues. Hence, the proper > way is to detect the architecture and not the opering system.
Doesn't that contradict with what the patch does? -#if defined __arm__ || defined __ARM__ || defined ARM || defined __ARM || defined __arm || defined __ARM_ARCH ||defined __aarch64__ ||( defined(__sun) && defined(__SVR4)) +#if defined __arm__ || defined __ARM__ || defined ARM || defined __ARM || defined __arm || defined __ARM_ARCH ||defined __aarch64__ ||defined __sparc__ ||( defined(__sun) && defined(__SVR4)) This merely adds a test for the SPARC architecture, but still keeps the special case for Solaris. Hence it will still enable alignment on Solaris/x86.
