[Mesa-dev] [PATCH 0/7] mesa: Enable -fstrict-aliasing

Erik Faye-Lund kusmabite at gmail.com
Tue Jun 28 08:51:09 UTC 2016


On Mon, Jun 27, 2016 at 11:42 PM, Matt Turner <mattst88 at gmail.com> wrote:
> Based on work by Davin McCall <davmac at davmac.org> from last summer.
>
> The biggest change is to exec_list. Previously, the head and tail sentinels
> overlapped, saving the size of a pointer. Unfortunately this is not allowed by
> the aliasing rules.
>
> I have fixed all warnings GCC reports in my normal build. I have no attempted
> to see what else needs to be fixed. I hope that the respective owners of the
> rest of Mesa can look into the remaining warnings.
>
> This series depends on my 4 patch series to glx, and the trivial "[PATCH] i965:
> Simplify foreach_inst_in_block_safe() macro."
>
> Discuss!

I like it. I have some similar patches here:
https://github.com/kusma/mesa/tree/strict-aliasing

I'm not entirely convinced about the endianess-correctness of all of
the memcpy-conversions, though... but I could easily be wrong.


More information about the mesa-dev mailing list