Re: [Mesa-dev] [PATCH 4/4] i965: Drop unnecessary bo->align field.

2018-03-26 Thread Jason Ekstrand
On March 26, 2018 22:39:33 Jason Ekstrand wrote: Some day we may need this because Ys tiled images must be 64K aligned. However, we can always put it back in. Rb If you're not worries about sparse right now (which I wouldn't be), you can put my rb on the other too. On March 26, 2018 16:5

Re: [Mesa-dev] [PATCH 4/4] i965: Drop unnecessary bo->align field.

2018-03-26 Thread Jason Ekstrand
Some day we may need this because Ys tiled images must be 64K aligned. However, we can always put it back in. Rb On March 26, 2018 16:52:42 Kenneth Graunke wrote: bo->align is always 0; there's no need to waste 8 bytes storing it. Thanks to C99 initializers zeroing fields, we can completely

[Mesa-dev] [PATCH 4/4] i965: Drop unnecessary bo->align field.

2018-03-26 Thread Kenneth Graunke
bo->align is always 0; there's no need to waste 8 bytes storing it. Thanks to C99 initializers zeroing fields, we can completely drop the only read of the field altogether. --- src/mesa/drivers/dri/i965/brw_bufmgr.c| 2 -- src/mesa/drivers/dri/i965/brw_bufmgr.h| 7 --- src/mesa