Reviewed-by: Alejandro Piñeiro <apinhe...@igalia.com>

On 22/03/18 01:58, Ian Romanick wrote:
> From: Ian Romanick <ian.d.roman...@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 about 900 more shaders.
>
> Signed-off-by: Ian Romanick <ian.d.roman...@intel.com>
> ---
>  src/intel/compiler/brw_fs_cmod_propagation.cpp | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/intel/compiler/brw_fs_cmod_propagation.cpp 
> b/src/intel/compiler/brw_fs_cmod_propagation.cpp
> index 4625d69..b995a51 100644
> --- a/src/intel/compiler/brw_fs_cmod_propagation.cpp
> +++ b/src/intel/compiler/brw_fs_cmod_propagation.cpp
> @@ -62,7 +62,8 @@ opt_cmod_propagation_local(const gen_device_info *devinfo, 
> 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;
>  

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

Reply via email to