[Mesa-dev] [PATCH 4/6] i965/vec4: Allow cmod propagation when src0 is a uniform or shader input

[Alejandro Piñeiro]

Reviewed-by: Alejandro Piñeiro <apinheiro at igalia.com>


On 22/03/18 01:58, Ian Romanick wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
>
> No shader-db changes.  This source must have been written by a previous
> instruction, so it cannot be a uniform or a shader input.  However, this
> change allows the next commit to help more shaders.
>
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> ---
>  src/intel/compiler/brw_vec4_cmod_propagation.cpp | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/intel/compiler/brw_vec4_cmod_propagation.cpp b/src/intel/compiler/brw_vec4_cmod_propagation.cpp
> index 0d72d82..7f1001b 100644
> --- a/src/intel/compiler/brw_vec4_cmod_propagation.cpp
> +++ b/src/intel/compiler/brw_vec4_cmod_propagation.cpp
> @@ -49,7 +49,8 @@ opt_cmod_propagation_local(bblock_t *block)
>             inst->opcode != BRW_OPCODE_MOV) ||
>            inst->predicate != BRW_PREDICATE_NONE ||
>            !inst->dst.is_null() ||
> -          inst->src[0].file != VGRF ||
> +          (inst->src[0].file != VGRF && inst->src[0].file != ATTR &&
> +           inst->src[0].file != UNIFORM) ||
>            inst->src[0].abs)
>           continue;
>