wine/vkd3d
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/vkd3d.git synced 2025-09-12 18:50:22 -07:00
Code Issues Packages Projects Releases Wiki Activity

tests: Stop probing all pixels when drawing a uniform colour.

Browse Source 
This is simply unnecessary and wastes time.

As part of this, simply remove the "all" directive. Only for a couple of tests
is it even potentially interesting to validate all pixels (e.g.
nointerpolation.shader_test), and for those "all" is replaced with an explicit
(0, 0, 640, 480) rect.

In all other cases we just probe (0, 0).
This commit is contained in:
Elizabeth Figura
2024-02-10 13:16:22 -06:00
committed by Henri Verbeet
parent 021eb28584
commit d3ba810c98
Notes: Henri Verbeet 2024-06-14 00:05:14 +02:00 
Approved-by: Giovanni Mascellani (@giomasce)
Approved-by: Henri Verbeet (@hverbeet)
Merge-Request: https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/908
163 changed files with 1003 additions and 1007 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
tests/hlsl/register-reservations-space.shader_test
View File

@@ -27,7 +27,7 @@ float4 main() : sv_target
[test]
todo(glsl) draw quad
probe all rgba (1, 1, 1, 99)
probe (0, 0) rgba (1, 1, 1, 99)
[pixel shader fail(sm>=6)]
@@ -40,7 +40,7 @@ float4 main() : sv_target
[test]
todo(glsl) draw quad
probe all rgba (1, 1, 1, 99)
probe (0, 0) rgba (1, 1, 1, 99)
[pixel shader fail todo]
@@ -62,7 +62,7 @@ float4 main() : sv_target
[test]
draw quad
probe all rgba (0, 0, 0, 99)
probe (0, 0) rgba (0, 0, 0, 99)
[pixel shader fail(sm<6)]
@@ -75,7 +75,7 @@ float4 main() : sv_target
[test]
draw quad
probe all rgba (0, 0, 0, 99)
probe (0, 0) rgba (0, 0, 0, 99)
% Specifying a profile is just broken. The first reservation (or, with sm6, the
@@ -91,8 +91,8 @@ float4 main() : sv_target
[test]
todo(sm<6) draw quad
if(sm>=6) probe all rgba (2, 2, 2, 99)
if(sm<6) probe all rgba (1, 1, 1, 99)
if(sm>=6) probe (0,0) rgba (2, 2, 2, 99)
if(sm<6) probe (0,0) rgba (1, 1, 1, 99)
% This actually inheres to 5.1+; it doesn't matter whether "space" is specified.
@@ -107,8 +107,8 @@ float4 main() : sv_target
[test]
todo(sm<6) draw quad
if(sm>=6) probe all rgba (2, 2, 2, 99)
if(sm<6) probe all rgba (1, 1, 1, 99)
if(sm>=6) probe (0,0) rgba (2, 2, 2, 99)
if(sm<6) probe (0,0) rgba (1, 1, 1, 99)
% It's still illegal to specify multiple contradictory reservations with the
@@ -135,8 +135,8 @@ float4 main() : sv_target
[test]
todo(sm<6) draw quad
if(sm>=6) probe all rgba (2, 2, 2, 99)
if(sm<6) probe all rgba (1, 1, 1, 99)
if(sm>=6) probe (0,0) rgba (2, 2, 2, 99)
if(sm<6) probe (0,0) rgba (1, 1, 1, 99)
% Test conflicts.