On Fri, Feb 6, 2015 at 3:02 AM, Ian Romanick <i...@freedesktop.org> wrote:
>> @@ -708,6 +763,9 @@ ir_expression::constant_expression_value(struct 
>> hash_table *variable_context)
>>        case GLSL_TYPE_FLOAT:
>>           data.f[c] = op[0]->value.f[c] - floor(op[0]->value.f[c]);
>>           break;
>> +      case GLSL_TYPE_DOUBLE:
>> +         data.d[c] = op[0]->value.d[c] - floor(op[0]->value.d[c]);
>> +         break;
>>        default:
>>           assert(0);
>
> Maybe follow-up by replacing a bunch of these with unreachable().

That seems highly unrelated to this series. Good newbie project though
-- audit all the assert(const)'s and convert into unreachable as
necessary. [I think there's basically no case where assert(const) is
right, in light of unreachable existing...]
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to