Re: [Mesa-dev] [PATCH] i965: Use a ZERO() macro to initialize GENX_* structs.

2017-07-14 Thread Emil Velikov
On 14 July 2017 at 20:13, Matt Turner wrote: > On Fri, Jul 14, 2017 at 11:15 AM, Rafael Antognolli > wrote: >> gen4 have commands which start with KernelStartPointer, which is a >> struct, so if we initialize it struct = { 0 }, we get warnings on some >> compilers. > > Which compilers? > GCC (pre

Re: [Mesa-dev] [PATCH] i965: Use a ZERO() macro to initialize GENX_* structs.

2017-07-14 Thread Matt Turner
On Fri, Jul 14, 2017 at 12:27 PM, Rafael Antognolli wrote: > On Fri, Jul 14, 2017 at 12:13:59PM -0700, Matt Turner wrote: >> On Fri, Jul 14, 2017 at 11:15 AM, Rafael Antognolli >> wrote: >> > gen4 have commands which start with KernelStartPointer, which is a >> > struct, so if we initialize it st

Re: [Mesa-dev] [PATCH] i965: Use a ZERO() macro to initialize GENX_* structs.

2017-07-14 Thread Rafael Antognolli
On Fri, Jul 14, 2017 at 12:13:59PM -0700, Matt Turner wrote: > On Fri, Jul 14, 2017 at 11:15 AM, Rafael Antognolli > wrote: > > gen4 have commands which start with KernelStartPointer, which is a > > struct, so if we initialize it struct = { 0 }, we get warnings on some > > compilers. > > Which co

Re: [Mesa-dev] [PATCH] i965: Use a ZERO() macro to initialize GENX_* structs.

2017-07-14 Thread Matt Turner
On Fri, Jul 14, 2017 at 11:15 AM, Rafael Antognolli wrote: > gen4 have commands which start with KernelStartPointer, which is a > struct, so if we initialize it struct = { 0 }, we get warnings on some > compilers. Which compilers? > > This macro works around that and will silent such warnings.

[Mesa-dev] [PATCH] i965: Use a ZERO() macro to initialize GENX_* structs.

2017-07-14 Thread Rafael Antognolli
gen4 have commands which start with KernelStartPointer, which is a struct, so if we initialize it struct = { 0 }, we get warnings on some compilers. This macro works around that and will silent such warnings. Signed-off-by: Rafael Antognolli Cc: Jason Ekstrand --- src/mesa/drivers/dri/i965/gen