[Mesa-dev] [PATCH 2/6] glsl: Add ir support for `sample` qualifier; adjust compiler and linker

Chris Forbes chrisf at ijw.co.nz
Fri Dec 6 13:02:51 PST 2013


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 <currojerez at riseup.net> wrote:
> Chris Forbes <chrisf at 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;
>>           }
>>        }
>>
>


More information about the mesa-dev mailing list