Re: [Mesa-dev] [PATCH] i965: Replace default case with list of enum values.

2015-10-31 Thread Matt Turner
On Fri, Oct 30, 2015 at 5:19 PM, Ian Romanick wrote: > On 10/29/2015 05:52 PM, Matt Turner wrote: >> If we add a new file type, we'd like to get warnings if it's not >> handled. >> >> Unfortuately, gcc seems to have bugs (see the XXX). > > Did you submit a GCC bug? I did some research and found

Re: [Mesa-dev] [PATCH] i965: Replace default case with list of enum values.

2015-10-30 Thread Ian Romanick
On 10/29/2015 05:52 PM, Matt Turner wrote: > If we add a new file type, we'd like to get warnings if it's not > handled. > > Unfortuately, gcc seems to have bugs (see the XXX). Did you submit a GCC bug? Assuming there are not warnings added, this patch is Reviewed-by: Ian Romanick > --- > sr

[Mesa-dev] [PATCH] i965: Replace default case with list of enum values.

2015-10-29 Thread Matt Turner
If we add a new file type, we'd like to get warnings if it's not handled. Unfortuately, gcc seems to have bugs (see the XXX). --- src/mesa/drivers/dri/i965/brw_fs.cpp | 16 +--- src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | 7 --- src/mesa/drivers/dri/i