[PATCH 1/6] make byte swapping macros less opaque to the optimizer

Matt Turner mattst88 at gmail.com
Fri May 13 14:24:29 PDT 2011


On Fri, May 13, 2011 at 4:43 PM, Keith Packard <keithp at keithp.com> wrote:
> On Fri, 29 Apr 2011 14:46:23 +0300, Siarhei Siamashka <siarhei.siamashka at gmail.com> wrote:
>
>> And of course, if none of the swapl/swaps/cpswapl/cpswaps are supposed
>> to deal with unaligned pointers, then this is not an issue at all.
>
> All of the protocol structures are aligned on 'natural' boundaries;
> swaps within those should never be unaligned.

Thanks for clearing that up.

So, after fixing the couple of unaligned accesses that Siarhei's trick
uncovered, the patch should be OK as-is.

I'll see about fixing those.

Matt


More information about the xorg-devel mailing list