Flyby comments...

On Sep 30, 2016 3:34 PM, "Anuj Phogat" <anuj.pho...@gmail.com> wrote:
>
> Signed-off-by: Anuj Phogat <anuj.pho...@gmail.com>
> ---
>  src/intel/vulkan/gen7_pipeline.c | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/src/intel/vulkan/gen7_pipeline.c
b/src/intel/vulkan/gen7_pipeline.c
> index 8d133b6..f98aef5 100644
> --- a/src/intel/vulkan/gen7_pipeline.c
> +++ b/src/intel/vulkan/gen7_pipeline.c
> @@ -121,6 +121,12 @@ genX(graphics_pipeline_create)(
>           };
>           vs.PerThreadScratchSpace      =
scratch_space(&vs_prog_data->base.base);
>
> +         vs.FloatingPointMode          =
vs_prog_data->base.base.use_alt_mode ?
> +                                         Alternate : IEEE754;

I'm pretty sure we only use alt mode for ARB programs do it doesn't really
apply to Vulkan.  Explicitly setting it to IEEE seems reasonable though.

> +         vs.SamplerCount               = 0;
> +         vs.BindingTableEntryCount     =
> +            vs_prog_data->base.base.binding_table.size_bytes / 4;

We don't set the binding table size field in the Vulkan driver so I don't
think this actually does anything.  Maybe we should actually set sampler
and surface state counts to something reasonable?  It might have a
performance impact but I don't know if we've ever measured it.

> +
>           vs.DispatchGRFStartRegisterforURBData    =
>              vs_prog_data->base.base.dispatch_grf_start_reg;
>
> --
> 2.5.5
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to