On Saturday, May 02, 2015 06:29:30 PM Francisco Jerez wrote:
> ---
> src/mesa/drivers/dri/i965/brw_ir_fs.h | 10 ++++++++++
> src/mesa/drivers/dri/i965/brw_ir_vec4.h | 9 +++++++++
> 2 files changed, 19 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_ir_fs.h
> b/src/mesa/drivers/dri/i965/brw_ir_fs.h
> index ce23fc5..6c65632 100644
> --- a/src/mesa/drivers/dri/i965/brw_ir_fs.h
> +++ b/src/mesa/drivers/dri/i965/brw_ir_fs.h
> @@ -258,4 +258,14 @@ public:
> bool pi_noperspective:1; /**< Pixel interpolator noperspective flag */
> };
>
> +/**
> + * Disable per-channel control flow execution masking on \p inst.
> + */
> +static inline fs_inst *
> +exec_all(fs_inst *inst)
> +{
> + inst->force_writemask_all = true;
> + return inst;
> +}
> +
> #endif
> diff --git a/src/mesa/drivers/dri/i965/brw_ir_vec4.h
> b/src/mesa/drivers/dri/i965/brw_ir_vec4.h
> index 36a8224..48dd90f 100644
> --- a/src/mesa/drivers/dri/i965/brw_ir_vec4.h
> +++ b/src/mesa/drivers/dri/i965/brw_ir_vec4.h
> @@ -192,6 +192,15 @@ public:
> }
> };
>
> +/**
> + * Disable per-channel control flow execution masking on \p inst.
> + */
> +inline vec4_instruction *
> +exec_all(vec4_instruction *inst)
> +{
> + inst->force_writemask_all = true;
> + return inst;
> +}
> } /* namespace brw */
>
> #endif
> Patches 3-6 are: Reviewed-by: Kenneth Graunke <[email protected]> Matt and I were confused about the "exec_" prefix in the functions though - why "exec"? "exec_all" makes a lot of sense here - execute on all channels - but "exec_saturate" seems a little odd to me. Perhaps we can find a different prefix (or you can convince us).
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
