Pekka Paalanen <ppaala...@gmail.com> writes: >> As a discussion point, wouldn't it be better for the ALIGN macro to >> assume 32-byte cache lines? Both 16-byte and 32-byte cachelines are >> currently in common use, but by aligning the buffers to 32-byte addresses >> we would simultaneously achieve 16-byte alignment.
It's not the size of cache lines that is interesting; it's the width of SIMD instructions. > I think Ben's explanation as seen in > https://patchwork.freedesktop.org/patch/49898/ > covers all Søren's concerns (it quotes everything Søren said about the > patch), and I see no reason to reject this. Well, the concern that SIMD destination fetchers (for example a 565 destination iterator) can no longer do a full SIMD write to the buffer is still there. It's not just combiners that write to the destination buffer. But whatever, it's not really worth arguing about. Let's just push this. We can fix it if SIMD destination fetchers ever actually happen. Søren _______________________________________________ Pixman mailing list Pixman@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/pixman