OK, I'd like to handle that [and any similar cases] in a follow-up
patch with some additional piglits.

-- Chris

On Sat, Dec 7, 2013 at 12:24 AM, Francisco Jerez <curroje...@riseup.net> wrote:
> Chris Forbes <chr...@ijw.co.nz> writes:
>>[...]
>> @@ -4662,6 +4673,7 @@ ast_process_structure_or_interface_block(exec_list 
>> *instructions,
>>           fields[i].interpolation =
>>              interpret_interpolation_qualifier(qual, var_mode, state, &loc);
>>           fields[i].centroid = qual->flags.q.centroid ? 1 : 0;
>> +         fields[i].sample = qual->flags.q.sample ? 1 : 0;
>
> Hi Chris, I just realized, we should probably make sure that this
> doesn't happen in a struct member declaration -- Only precision
> qualifiers are allowed in them according to the GL spec.
>
>>
>>           if (qual->flags.q.row_major || qual->flags.q.column_major) {
>>              if (!qual->flags.q.uniform) {
>> @@ -4930,6 +4942,8 @@ ast_interface_block::hir(exec_list *instructions,
>>                 earlier_per_vertex->fields.structure[j].interpolation;
>>              fields[i].centroid =
>>                 earlier_per_vertex->fields.structure[j].centroid;
>> +            fields[i].sample =
>> +               earlier_per_vertex->fields.structure[j].sample;
>>           }
>>        }
>>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to