<p dir="ltr"></p>
<p dir="ltr">On Sep 30, 2016 3:34 PM, "Anuj Phogat" <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> wrote:<br>
><br>
> Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>><br>
> ---<br>
>  src/intel/vulkan/gen7_pipeline.c | 4 ++++<br>
>  1 file changed, 4 insertions(+)<br>
><br>
> diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c<br>
> index 9411631..4bab0bc 100644<br>
> --- a/src/intel/vulkan/gen7_pipeline.c<br>
> +++ b/src/intel/vulkan/gen7_pipeline.c<br>
> @@ -243,6 +243,10 @@ genX(graphics_pipeline_create)(<br>
>           /* Haswell requires the sample mask to be set in this packet as well as<br>
>            * in 3DSTATE_SAMPLE_MASK; the values should match. */<br>
>           /* _NEW_BUFFERS, _NEW_MULTISAMPLE */<br>
> +#        if (GEN_IS_HASWELL)</p>
<p dir="ltr">There is some code that uses this #if style but I think the "official" style is to not indent and not use the parens.</p>
<p dir="ltr">> +         ps.SampleMask                    = 0xff;<br>
> +#        endif<br>
> +         ps.SamplerCount                  = 1;</p>
<p dir="ltr">Of we're going to add code to set sampler count, it should be accurate.  It's probably best to drop this line and have a later patch which actually sets it correctly.</p>
<p dir="ltr">>        }<br>
><br>
>        uint32_t samples = pCreateInfo->pMultisampleState ?<br>
> --<br>
> 2.5.5<br>
><br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br></p>