On Fri, Feb 6, 2015 at 3:29 AM, Pohjolainen, Topi
<topi.pohjolai...@intel.com> wrote:
> On Fri, Feb 06, 2015 at 10:04:13AM +0200, Pohjolainen, Topi wrote:
>> On Thu, Feb 05, 2015 at 11:56:28PM -0500, Ilia Mirkin wrote:
>> > From: Dave Airlie <airl...@gmail.com>
>> >
>> > This causes a lot of warnings about unchecked type in
>> > switch statements - fix them later.
>>
>> The rest of the series fixes things in the compiler frontend but leaves
>> a lot unchecked in the compiler backend (at least in i965). Also we now
>> get complains in NIR.
>> Putting something intermediate to silence things that get fixed by the
>> series itself probably doesn't make sense but I wonder if we should
>> address NIR at least. Thoughts?
>
> Furthermore this patch introduces warnings in the uniform module test that
> are not fixed by the rest of the series either:
>
> tests/uniform_initializer_utils.cpp: In function 'void
> generate_data_element(void*, const glsl_type*, ir_constant*&, unsigned int)':
> tests/uniform_initializer_utils.cpp:83:14: warning: enumeration value
> 'GLSL_TYPE_DOUBLE' not handled in switch [-Wswitch]
> switch (type->base_type) {
> ^
> tests/uniform_initializer_utils.cpp:83:14: warning: enumeration value
> 'GLSL_TYPE_IMAGE' not handled in switch [-Wswitch]
> tests/uniform_initializer_utils.cpp:111:14: warning: enumeration value
> 'GLSL_TYPE_DOUBLE' not handled in switch [-Wswitch]
> switch (type->base_type) {
> ^
> tests/uniform_initializer_utils.cpp:111:14: warning: enumeration value
> 'GLSL_TYPE_IMAGE' not handled in switch [-Wswitch]
> tests/uniform_initializer_utils.cpp: In function 'void
> verify_data(gl_constant_value*, unsigned int, ir_constant*, unsigned int,
> unsigned int)':
> tests/uniform_initializer_utils.cpp:211:10: warning: enumeration value
> 'GLSL_TYPE_DOUBLE' not handled in switch [-Wswitch]
> switch (val->type->base_type) {
> ^
> tests/uniform_initializer_utils.cpp:211:10: warning: enumeration value
> 'GLSL_TYPE_IMAGE' not handled in switch [-Wswitch]
>
I've added a note to myself to fix these up, thanks!
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
