Hi Marek,

On 20 June 2017 at 19:00, Marek Olšák <[email protected]> wrote:
> From: Marek Olšák <[email protected]>
>
> sizeof(struct si_shader_key):
>   Before reverting the 2 commits: 120 bytes
>   After reverting the 2 commits: 128 bytes
>   With #pragma pack: 107 bytes
>
> I'm not sure if memcmp with a byte-aligned size is a good idea.

If you guys decide to go with this, please consider annotating the
individual structs.
Otherwise it's likely that one will add/remove a struct between the
pragma push/pop, causing some interesting and annoying to debug
issues.

-Emil
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to