Am 10.01.2018 um 07:48 schrieb Dave Airlie:
> From: Dave Airlie <airl...@redhat.com>
> 
> This stops them being optimised out.
> ---
>  src/gallium/drivers/r600/sb/sb_bc_parser.cpp | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/src/gallium/drivers/r600/sb/sb_bc_parser.cpp 
> b/src/gallium/drivers/r600/sb/sb_bc_parser.cpp
> index ae92a767b4..de3984f596 100644
> --- a/src/gallium/drivers/r600/sb/sb_bc_parser.cpp
> +++ b/src/gallium/drivers/r600/sb/sb_bc_parser.cpp
> @@ -149,11 +149,14 @@ int bc_parser::parse_decls() {
>               }
>       }
>  
> -     if (sh->target == TARGET_VS || sh->target == TARGET_ES || sh->target == 
> TARGET_HS)
> +     if (sh->target == TARGET_VS || sh->target == TARGET_ES || sh->target == 
> TARGET_HS || sh->target == TARGET_LS)
>               sh->add_input(0, 1, 0x0F);
>       else if (sh->target == TARGET_GS) {
>               sh->add_input(0, 1, 0x0F);
>               sh->add_input(1, 1, 0x0F);
> +     } else if (sh->target == TARGET_COMPUTE) {
> +             sh->add_input(0, 1, 0x0F);
> +             sh->add_input(1, 1, 0x0F);
For compute, won't you never get here because the !pshader clause
above will catch it?
By the looks of it, !pshader clause would catch both fetch and compute
shaders, and add add 2 preloaded regs, albeit for fetch shader I have no
idea why it would reserve 2 (I think should be same as vs, just 1).

But otherwise looks good.
For 0,1,2
Reviewed-by: Roland Scheidegger <srol...@vmware.com>

>       }
>  
>       bool ps_interp = ctx.hw_class >= HW_CLASS_EVERGREEN
> 
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to