[Piglit] [PATCH] multisample_blit_scaled: Change 8X MSAA sample mapping

Jason Ekstrand jason at jlekstrand.net
Fri Aug 12 01:11:27 UTC 2016


LGTM R-B

On Aug 11, 2016 12:30 PM, "Anuj Phogat" <anuj.phogat at gmail.com> wrote:

> This is required following a change in 8X multisample positions
> in i965 driver.
>
> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
> ---
>  .../ext_framebuffer_multisample_blit_scaled/blit-scaled.cpp    | 10
> +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/tests/spec/ext_framebuffer_multisample_blit_scaled/blit-scaled.cpp
> b/tests/spec/ext_framebuffer_multisample_blit_scaled/blit-scaled.cpp
> index 927615d..90e96ef 100644
> --- a/tests/spec/ext_framebuffer_multisample_blit_scaled/blit-scaled.cpp
> +++ b/tests/spec/ext_framebuffer_multisample_blit_scaled/blit-scaled.cpp
> @@ -162,13 +162,13 @@ compile_shader(GLenum target)
>          *
>          * 8X MSAA sample index layout    8x MSAA sample number layout
>          *           ---------                      ---------
> -        *           | 0 | 1 |                      | 5 | 2 |
> +        *           | 0 | 1 |                      | 3 | 7 |
>          *           ---------                      ---------
> -        *           | 2 | 3 |                      | 4 | 6 |
> +        *           | 2 | 3 |                      | 5 | 0 |
>          *           ---------                      ---------
> -        *           | 4 | 5 |                      | 0 | 3 |
> +        *           | 4 | 5 |                      | 1 | 2 |
>          *           ---------                      ---------
> -        *           | 6 | 7 |                      | 7 | 1 |
> +        *           | 6 | 7 |                      | 4 | 6 |
>          *           ---------                      ---------
>          *
>          * 16X MSAA sample index layout  16x MSAA sample number layout
> @@ -190,7 +190,7 @@ compile_shader(GLenum target)
>                 sample_number =  "int(2 * fract(coord.x) + 4 *
> fract(coord.y))";
>         break;
>         case 8:
> -               sample_map = "  const int sample_map[8] = int[8](5 , 2, 4,
> 6, 0, 3, 7, 1);\n";
> +               sample_map = "  const int sample_map[8] = int[8](3, 7, 5,
> 0, 1, 2, 4, 6);\n";
>                 sample_number = "sample_map[int(2 * fract(coord.x) + 8 *
> fract(coord.y))]";
>                 break;
>         case 16:
> --
> 2.5.5
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/piglit/attachments/20160811/f9af623c/attachment.html>


More information about the Piglit mailing list