On Mon, Oct 5, 2015 at 6:29 AM, Neil Roberts <n...@linux.intel.com> wrote:
> It is possible to directly predicate the WHILE instruction. In this
> case there will be a second successor block because the execution can
> resume from the instruction after the loop. This will be used in a
> subsequent patch.
> ---
>  src/mesa/drivers/dri/i965/brw_cfg.cpp | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_cfg.cpp 
> b/src/mesa/drivers/dri/i965/brw_cfg.cpp
> index 91d53ef..531fa16 100644
> --- a/src/mesa/drivers/dri/i965/brw_cfg.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_cfg.cpp
> @@ -305,6 +305,10 @@ cfg_t::cfg_t(exec_list *instructions)
>
>           assert(cur_do != NULL && cur_while != NULL);
>          cur->add_successor(mem_ctx, cur_do);
> +
> +         if (inst->predicate)
> +            cur->add_successor(mem_ctx, cur_while);
> +

Reviewed-by: Matt Turner <matts...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to