[Mesa-dev] [PATCH 1/3] glsl: remove unused field from ir_call

Thomas Helland thomashelland90 at gmail.com
Fri Aug 4 12:15:15 UTC 2017


This patch is:

Reviewed-by: Thomas Helland <thomashelland90 at gmail.com>

2017-08-04 7:25 GMT+00:00 Timothy Arceri <tarceri at itsqueeze.com>:
> ---
>  src/compiler/glsl/ir.h | 5 -----
>  1 file changed, 5 deletions(-)
>
> diff --git a/src/compiler/glsl/ir.h b/src/compiler/glsl/ir.h
> index 40f3338470..d425b97aca 100644
> --- a/src/compiler/glsl/ir.h
> +++ b/src/compiler/glsl/ir.h
> @@ -1595,32 +1595,30 @@ public:
>   */
>  class ir_call : public ir_instruction {
>  public:
>     ir_call(ir_function_signature *callee,
>            ir_dereference_variable *return_deref,
>            exec_list *actual_parameters)
>        : ir_instruction(ir_type_call), return_deref(return_deref), callee(callee), sub_var(NULL), array_idx(NULL)
>     {
>        assert(callee->return_type != NULL);
>        actual_parameters->move_nodes_to(& this->actual_parameters);
> -      this->use_builtin = callee->is_builtin();
>     }
>
>     ir_call(ir_function_signature *callee,
>            ir_dereference_variable *return_deref,
>            exec_list *actual_parameters,
>            ir_variable *var, ir_rvalue *array_idx)
>        : ir_instruction(ir_type_call), return_deref(return_deref), callee(callee), sub_var(var), array_idx(array_idx)
>     {
>        assert(callee->return_type != NULL);
>        actual_parameters->move_nodes_to(& this->actual_parameters);
> -      this->use_builtin = callee->is_builtin();
>     }
>
>     virtual ir_call *clone(void *mem_ctx, struct hash_table *ht) const;
>
>     virtual ir_constant *constant_expression_value(struct hash_table *variable_context = NULL);
>
>     virtual void accept(ir_visitor *v)
>     {
>        v->visit(this);
>     }
> @@ -1648,23 +1646,20 @@ public:
>     ir_dereference_variable *return_deref;
>
>     /**
>      * The specific function signature being called.
>      */
>     ir_function_signature *callee;
>
>     /* List of ir_rvalue of paramaters passed in this call. */
>     exec_list actual_parameters;
>
> -   /** Should this call only bind to a built-in function? */
> -   bool use_builtin;
> -
>     /*
>      * ARB_shader_subroutine support -
>      * the subroutine uniform variable and array index
>      * rvalue to be used in the lowering pass later.
>      */
>     ir_variable *sub_var;
>     ir_rvalue *array_idx;
>  };
>
>
> --
> 2.13.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list