On Tuesday, December 08, 2015 01:46:22 PM Jason Ekstrand wrote:
> diff --git a/src/glsl/nir/nir_lower_io.c b/src/glsl/nir/nir_lower_io.c
> index f64ac69..a2723d5 100644
> --- a/src/glsl/nir/nir_lower_io.c
> +++ b/src/glsl/nir/nir_lower_io.c
> @@ -333,18 +311,18 @@ nir_lower_io(nir_shader *shader, nir_variable_mode mode,
>   * Return the indirect source for a load/store indirect intrinsic.

This comment could use updating.  How about:

/**
 * Return the offset soruce for a load/store intrinsic.
 */

>   */
>  nir_src *
> -nir_get_io_indirect_src(nir_intrinsic_instr *instr)
> +nir_get_io_offset_src(nir_intrinsic_instr *instr)
>  {
>     switch (instr->intrinsic) {
> -   case nir_intrinsic_load_input_indirect:
> -   case nir_intrinsic_load_output_indirect:
> -   case nir_intrinsic_load_uniform_indirect:
> +   case nir_intrinsic_load_input:
> +   case nir_intrinsic_load_output:
> +   case nir_intrinsic_load_uniform:
>        return &instr->src[0];
> -   case nir_intrinsic_load_per_vertex_input_indirect:
> -   case nir_intrinsic_load_per_vertex_output_indirect:
> -   case nir_intrinsic_store_output_indirect:
> +   case nir_intrinsic_load_per_vertex_input:
> +   case nir_intrinsic_load_per_vertex_output:
> +   case nir_intrinsic_store_output:
>        return &instr->src[1];
> -   case nir_intrinsic_store_per_vertex_output_indirect:
> +   case nir_intrinsic_store_per_vertex_output:
>        return &instr->src[2];
>     default:
>        return NULL;

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to