[PATCH v5 0/4] MR616: tests: Improvements for SM1 testing.
This is the first part of !608, which I decided to separate to ease review. It consists of improvements to SM1 testing, mainly: - Correctly passing int and bool uniforms as IEEE 754 floats to SM1 profiles. - The introduction of the "only" qualifier to avoid abusing `[require]` blocks in case tests retrieve different results for different shader models. - Allowing some of the tests we already have for SM4/SM6 to run in SM1 after those improvements. -- v5: tests: Remove [require] directives for tests that use int and bool uniforms. tests/shader-runner: Introduce "if" qualifier. tests: Don't ignore SM1 on a non-const-indexing.shader_test test. tests/vkd3d-shader: Set uninitialized uniforms to a value. https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/616
From: Francisco Casas <fcasas(a)codeweavers.com> Avoids a potential source of non-determinism because the padding bytes between uniforms could be left uninitialized. --- tests/shader_runner.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/shader_runner.c b/tests/shader_runner.c index fe5ee5972..866fd5d4a 100644 --- a/tests/shader_runner.c +++ b/tests/shader_runner.c @@ -551,9 +551,13 @@ static void set_resource(struct shader_runner *runner, struct resource *resource static void set_uniforms(struct shader_runner *runner, size_t offset, size_t count, const void *uniforms) { + size_t initial_count = runner->uniform_count; + runner->uniform_count = align(max(runner->uniform_count, offset + count), 4); vkd3d_array_reserve((void **)&runner->uniforms, &runner->uniform_capacity, runner->uniform_count, sizeof(*runner->uniforms)); + memset(runner->uniforms + initial_count, 127, + (runner->uniform_count - initial_count) * sizeof(*runner->uniforms)); memcpy(runner->uniforms + offset, uniforms, count * sizeof(*runner->uniforms)); } -- GitLab https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/616
From: Francisco Casas <fcasas(a)codeweavers.com> The previous [require] block makes us skip the test for SM4. --- tests/hlsl/non-const-indexing.shader_test | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/hlsl/non-const-indexing.shader_test b/tests/hlsl/non-const-indexing.shader_test index b0452148e..ba60367fe 100644 --- a/tests/hlsl/non-const-indexing.shader_test +++ b/tests/hlsl/non-const-indexing.shader_test @@ -216,7 +216,11 @@ draw quad probe all rgba (1, 5, 3, 4) -[pixel shader] +[require] +% reset requirements + + +[pixel shader todo(sm<4)] uniform float4 f[4]; uniform uint4 u; uniform uint4 v; -- GitLab https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/616
From: Francisco Casas <fcasas(a)codeweavers.com> When the "if" qualifier is added to a directive, the directive is skipped if the shader->minimum_shader_model is not included in the range. This can be used on the "probe" directives for tests that have different expected results on different shader models, without having to resort to [require] blocks. --- tests/hlsl/arithmetic-int.shader_test | 8 ++--- tests/hlsl/duplicate-modifiers.shader_test | 7 ++-- .../initializer-implicit-array.shader_test | 9 ++--- tests/hlsl/initializer-numeric.shader_test | 6 ++-- tests/hlsl/non-const-indexing.shader_test | 36 +++---------------- tests/shader_runner.c | 36 +++++++++++++++++-- 6 files changed, 47 insertions(+), 55 deletions(-) diff --git a/tests/hlsl/arithmetic-int.shader_test b/tests/hlsl/arithmetic-int.shader_test index 46b641811..7a64894d1 100644 --- a/tests/hlsl/arithmetic-int.shader_test +++ b/tests/hlsl/arithmetic-int.shader_test @@ -104,10 +104,9 @@ float4 main() : SV_TARGET draw quad probe all rgba (0.0, 0.0, 0.0, 0.0) + [require] shader model >= 4.0 -% dxcompiler performs this calculation on unsigned values and emits zero. -shader model < 6.0 [pixel shader] float4 main() : SV_TARGET @@ -120,10 +119,9 @@ float4 main() : SV_TARGET [test] draw quad -probe all rgba (-2147483648.0, -2147483648.0, -2147483648.0, -2147483648.0) +if(sm<6) probe all rgba (-2147483648.0, -2147483648.0, -2147483648.0, -2147483648.0) +if(sm>=6) probe all rgba (0.0, 0.0, 0.0, 0.0) -[require] -shader model >= 4.0 [pixel shader] float4 main() : sv_target diff --git a/tests/hlsl/duplicate-modifiers.shader_test b/tests/hlsl/duplicate-modifiers.shader_test index bf1d9c1b8..87f564952 100644 --- a/tests/hlsl/duplicate-modifiers.shader_test +++ b/tests/hlsl/duplicate-modifiers.shader_test @@ -1,7 +1,3 @@ -% Returns (0.1, 0.3, 0.2, 0.4) with dxcompiler -[require] -shader model < 6.0 - [pixel shader] typedef const precise row_major float2x2 mat_t; float4 main() : sv_target @@ -12,4 +8,5 @@ float4 main() : sv_target [test] draw quad -probe all rgba (0.1, 0.2, 0.3, 0.4) +if(sm<6) probe all rgba (0.1, 0.2, 0.3, 0.4) +if(sm>=6) probe all rgba (0.1, 0.3, 0.2, 0.4) diff --git a/tests/hlsl/initializer-implicit-array.shader_test b/tests/hlsl/initializer-implicit-array.shader_test index 25cd15644..e11825996 100644 --- a/tests/hlsl/initializer-implicit-array.shader_test +++ b/tests/hlsl/initializer-implicit-array.shader_test @@ -11,10 +11,6 @@ draw quad probe all rgba (50, 60, 70, 80) -% dxcompiler emits a nop shader which returns immediately. -[require] -shader model < 6.0 - [pixel shader] float4 main() : sv_target { @@ -26,10 +22,9 @@ float4 main() : sv_target [test] draw quad -probe all rgba (5.0, 6.0, 7.0, 8.0) +% dxcompiler emits a nop shader which returns immediately. +if(sm<6) probe all rgba (5.0, 6.0, 7.0, 8.0) -[require] -% reset requirements [pixel shader] float4 main() : sv_target diff --git a/tests/hlsl/initializer-numeric.shader_test b/tests/hlsl/initializer-numeric.shader_test index 617b67405..1b41dfb5a 100644 --- a/tests/hlsl/initializer-numeric.shader_test +++ b/tests/hlsl/initializer-numeric.shader_test @@ -60,9 +60,6 @@ draw quad probe all rgba (3.0, 250.0, 16.0, 4.2949673e+009) 4 -[require] -shader model < 6.0 - [pixel shader] float4 main() : sv_target { @@ -73,4 +70,5 @@ float4 main() : sv_target [test] draw quad -probe all rgba (-1294967296.0, 3000000000.0, 0.0, 0.0) 4 +if(sm<6) probe all rgba (-1294967296.0, 3000000000.0, 0.0, 0.0) 4 +if(sm>=6) probe all rgba (3000000000.0, 3000000000.0, 0.0, 0.0) 4 diff --git a/tests/hlsl/non-const-indexing.shader_test b/tests/hlsl/non-const-indexing.shader_test index ba60367fe..aed33092c 100644 --- a/tests/hlsl/non-const-indexing.shader_test +++ b/tests/hlsl/non-const-indexing.shader_test @@ -236,23 +236,6 @@ float4 main() : sv_target } % FXC is incapable of compiling this correctly, but results differ for SM1-3 vs SM4-5. -[require] -shader model < 4.0 - -[test] -uniform 0 float 1.0 -uniform 4 float 2.0 -uniform 8 float 3.0 -uniform 12 float 4.0 -uniform 16 uint4 3 1 0 2 -uniform 20 uint4 0 3 1 2 -todo draw quad -todo(sm<4) probe all rgba (1.0, 1.0, 1.0, 1.0) - -[require] -shader model >= 4.0 -shader model < 6.0 - [test] uniform 0 float 1.0 uniform 4 float 2.0 @@ -260,18 +243,7 @@ uniform 8 float 3.0 uniform 12 float 4.0 uniform 16 uint4 3 1 0 2 uniform 20 uint4 0 3 1 2 -draw quad -todo probe all rgba (4.0, 4.0, 4.0, 4.0) - -[require] -shader model >= 6.0 - -[test] -uniform 0 float 1.0 -uniform 4 float 2.0 -uniform 8 float 3.0 -uniform 12 float 4.0 -uniform 16 uint4 3 1 0 2 -uniform 20 uint4 0 3 1 2 -draw quad -probe all rgba (4.0, 3.0, 2.0, 1.0) +todo(sm<4) draw quad +if(sm<4) todo probe all rgba (1.0, 1.0, 1.0, 1.0) +if(sm>=4 & sm<6) todo probe all rgba (4.0, 4.0, 4.0, 4.0) +if(sm>=6) probe all rgba (4.0, 3.0, 2.0, 1.0) diff --git a/tests/shader_runner.c b/tests/shader_runner.c index 866fd5d4a..a683866c2 100644 --- a/tests/shader_runner.c +++ b/tests/shader_runner.c @@ -657,14 +657,46 @@ static void read_uint64_t2(const char **line, struct u64vec2 *v) static void parse_test_directive(struct shader_runner *runner, const char *line) { + bool skip_directive = false; + const char *line_ini; + bool match = true; char *rest; int ret; runner->is_todo = false; - if (match_string_with_args(line, "todo", &line, runner->minimum_shader_model)) + while (match) { - runner->is_todo = true; + match = false; + + if (match_string_with_args(line, "todo", &line, runner->minimum_shader_model)) + { + runner->is_todo = true; + match = true; + } + + line_ini = line; + if (match_string_with_args(line, "if", &line, runner->minimum_shader_model)) + { + match = true; + } + else if (line != line_ini) + { + /* Matched "if" but for other shader models. */ + skip_directive = true; + match = true; + } + } + + if (skip_directive) + { + const char *new_line; + + if ((new_line = strchr(line, '\n'))) + line = new_line + 1; + else + line += strlen(line); + return; } if (match_string(line, "dispatch", &line)) -- GitLab https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/616
From: Francisco Casas <fcasas(a)codeweavers.com> These tests should actually compile and run in SM1, which is possible if we pass the int and uint uniforms in the expected IEEE 754 float format for SM1 shaders. Also, bools should be passed as 1.0f or 0.0f to SM1. --- tests/hlsl/any.shader_test | 46 ++++---- tests/hlsl/bool-cast.shader_test | 18 ++- tests/hlsl/cast-to-float.shader_test | 21 ++-- tests/hlsl/cast-to-half.shader_test | 21 ++-- tests/hlsl/cast-to-int.shader_test | 24 ++-- tests/hlsl/cast-to-uint.shader_test | 4 +- tests/hlsl/ceil.shader_test | 10 +- tests/hlsl/floor.shader_test | 9 +- tests/hlsl/function-cast.shader_test | 7 +- tests/hlsl/ldexp.shader_test | 12 +- tests/hlsl/lerp.shader_test | 15 +-- tests/hlsl/sign.shader_test | 35 +++--- tests/hlsl/switch.shader_test | 169 ++++++++++++++++----------- tests/hlsl/trunc.shader_test | 9 +- 14 files changed, 222 insertions(+), 178 deletions(-) diff --git a/tests/hlsl/any.shader_test b/tests/hlsl/any.shader_test index afaf81fac..83a1dad97 100644 --- a/tests/hlsl/any.shader_test +++ b/tests/hlsl/any.shader_test @@ -48,10 +48,8 @@ uniform 0 float4 -1.0 0.0 0.0 0.0 todo(sm<4) draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) -[require] -shader model >= 4.0 -[pixel shader] +[pixel shader todo(sm<4)] uniform uint4 b; float4 main() : sv_target @@ -60,26 +58,32 @@ float4 main() : sv_target } [test] -uniform 0 uint4 1 1 1 1 -draw quad +if(sm<4) uniform 0 float4 1 1 1 1 +if(sm>=4) uniform 0 uint4 1 1 1 1 +todo(sm<4) draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) -uniform 0 uint4 1 0 0 0 -draw quad +if(sm<4) uniform 0 float4 1 0 0 0 +if(sm>=4) uniform 0 uint4 1 0 0 0 +todo(sm<4) draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) -uniform 0 uint4 0 1 0 0 -draw quad +if(sm<4) uniform 0 float4 0 1 0 0 +if(sm>=4) uniform 0 uint4 0 1 0 0 +todo(sm<4) draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) -uniform 0 uint4 0 0 1 0 -draw quad +if(sm<4) uniform 0 float4 0 0 1 0 +if(sm>=4) uniform 0 uint4 0 0 1 0 +todo(sm<4) draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) -uniform 0 uint4 0 0 0 1 -draw quad +if(sm<4) uniform 0 float4 0 0 0 1 +if(sm>=4) uniform 0 uint4 0 0 0 1 +todo(sm<4) draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) -uniform 0 uint4 0 0 0 0 -draw quad +if(sm<4) uniform 0 float4 0 0 0 0 +if(sm>=4) uniform 0 uint4 0 0 0 0 +todo(sm<4) draw quad probe all rgba (0.0, 0.0, 0.0, 0.0) -[pixel shader] +[pixel shader todo(sm<4)] uniform uint b; float4 main() : sv_target @@ -88,9 +92,11 @@ float4 main() : sv_target } [test] -uniform 0 uint4 1 0 0 0 -draw quad +if(sm<4) uniform 0 float4 1 0 0 0 +if(sm>=4) uniform 0 uint4 1 0 0 0 +todo(sm<4) draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) -uniform 0 uint4 0 0 0 0 -draw quad +if(sm<4) uniform 0 float4 0 0 0 0 +if(sm>=4) uniform 0 uint4 0 0 0 0 +todo(sm<4) draw quad probe all rgba (0.0, 0.0, 0.0, 0.0) diff --git a/tests/hlsl/bool-cast.shader_test b/tests/hlsl/bool-cast.shader_test index 09ca12e2b..f3e4cf0d6 100644 --- a/tests/hlsl/bool-cast.shader_test +++ b/tests/hlsl/bool-cast.shader_test @@ -14,11 +14,7 @@ draw quad probe all rgba (0.0, 0.0, 1.0, 1.0) -[require] -shader model >= 4.0 - - -[pixel shader] +[pixel shader todo(sm<4)] uniform float4 x; uniform int4 y; @@ -29,12 +25,13 @@ float4 main() : SV_TARGET [test] uniform 0 float4 0.0 0.0 2.0 4.0 -uniform 4 int4 0 1 0 10 -draw quad +if(sm<4) uniform 4 float4 0 1 0 10 +if(sm>=4) uniform 4 int4 0 1 0 10 +todo(sm<4) draw quad probe all rgba (0.0, 10.0, 1.0, 11.0) -[pixel shader] +[pixel shader todo(sm<4)] uniform bool4 b; float4 main() : sv_target @@ -43,6 +40,7 @@ float4 main() : sv_target } [test] -uniform 0 uint4 0x00000001 0x00000002 0x80000000 0x00000000 -draw quad +if(sm<4) uniform 0 float4 1.0 1.0 1.0 0.0 +if(sm>=4) uniform 0 uint4 0x00000001 0x00000002 0x80000000 0x00000000 +todo(sm<4) draw quad probe all rgba (2.0, 2.0, 2.0, 0.0) diff --git a/tests/hlsl/cast-to-float.shader_test b/tests/hlsl/cast-to-float.shader_test index caaf98c02..bcb546e2d 100644 --- a/tests/hlsl/cast-to-float.shader_test +++ b/tests/hlsl/cast-to-float.shader_test @@ -1,7 +1,4 @@ -[require] -shader model >= 4.0 - -[pixel shader] +[pixel shader todo(sm<4)] uniform int i; uniform uint u; uniform bool b; @@ -13,15 +10,19 @@ float4 main() : sv_target } [test] -uniform 0 int -1 -uniform 1 uint 3 -uniform 2 int -2 -uniform 3 float 0.5 -draw quad +if(sm<4) uniform 0 float -1 +if(sm<4) uniform 4 float 3 +if(sm<4) uniform 8 float 1 +if(sm<4) uniform 12 float 0.5 +if(sm>=4) uniform 0 int -1 +if(sm>=4) uniform 1 uint 3 +if(sm>=4) uniform 2 int -2 +if(sm>=4) uniform 3 float 0.5 +todo(sm<4) draw quad probe all rgba (0.5, 0.5, 0.5, 0.5) -[pixel shader] +[pixel shader] float4 main() : sv_target { int i = -1; diff --git a/tests/hlsl/cast-to-half.shader_test b/tests/hlsl/cast-to-half.shader_test index b8feb6760..0580fe25f 100644 --- a/tests/hlsl/cast-to-half.shader_test +++ b/tests/hlsl/cast-to-half.shader_test @@ -1,7 +1,4 @@ -[require] -shader model >= 4.0 - -[pixel shader] +[pixel shader todo(sm<4)] uniform int i; uniform uint u; uniform bool b; @@ -13,15 +10,19 @@ float4 main() : sv_target } [test] -uniform 0 int -1 -uniform 1 uint 3 -uniform 2 int -2 -uniform 3 float 0.5 -draw quad +if(sm<4) uniform 0 float -1 +if(sm<4) uniform 4 float 3 +if(sm<4) uniform 8 float 1 +if(sm<4) uniform 12 float 0.5 +if(sm>=4) uniform 0 int -1 +if(sm>=4) uniform 1 uint 3 +if(sm>=4) uniform 2 int -2 +if(sm>=4) uniform 3 float 0.5 +todo(sm<4) draw quad probe all rgba (0.5, 0.5, 0.5, 0.5) -[pixel shader] +[pixel shader] float4 main() : sv_target { int i = -1; diff --git a/tests/hlsl/cast-to-int.shader_test b/tests/hlsl/cast-to-int.shader_test index 3e850fb5b..385e3375f 100644 --- a/tests/hlsl/cast-to-int.shader_test +++ b/tests/hlsl/cast-to-int.shader_test @@ -1,7 +1,4 @@ -[require] -shader model >= 4.0 - -[pixel shader] +[pixel shader todo(sm<4)] uniform float f; uniform uint u; uniform bool b; @@ -19,15 +16,19 @@ float4 main() : sv_target } [test] -uniform 0 float 2.6 -uniform 1 int -2 -uniform 2 int -2 -uniform 3 float -3.6 -draw quad +if(sm<4) uniform 0 float 2.6 +if(sm<4) uniform 4 float -2 +if(sm<4) uniform 8 float 1.0 +if(sm<4) uniform 12 float -3.6 +if(sm>=4) uniform 0 float 2.6 +if(sm>=4) uniform 1 int -2 +if(sm>=4) uniform 2 int -2 +if(sm>=4) uniform 3 float -3.6 +todo(sm<4) draw quad probe all rgba (0.5, 0.5, 0.5, 0.5) -[pixel shader] +[pixel shader] float4 main() : sv_target { float f = 2.6; @@ -45,4 +46,5 @@ float4 main() : sv_target [test] draw quad -probe all rgba (0.5, 0.5, 0.5, 0.5) +if(sm<4) todo probe all rgba (0.5, 4.2949673e+009, 0.5, 0.5) +if(sm>=4) probe all rgba (0.5, 0.5, 0.5, 0.5) diff --git a/tests/hlsl/cast-to-uint.shader_test b/tests/hlsl/cast-to-uint.shader_test index 07479984a..9344fffc8 100644 --- a/tests/hlsl/cast-to-uint.shader_test +++ b/tests/hlsl/cast-to-uint.shader_test @@ -1,6 +1,8 @@ +% On SM1, uints can only be used with known-positive values. [require] shader model >= 4.0 + [pixel shader] uniform float f; uniform int i; @@ -26,8 +28,8 @@ uniform 3 float -3.6 draw quad probe all rgba (0.5, 0.5, 0.5, 0.5) -[pixel shader] +[pixel shader] float4 main() : sv_target { float f = 2.6; diff --git a/tests/hlsl/ceil.shader_test b/tests/hlsl/ceil.shader_test index 46414a92b..aa4bdf297 100644 --- a/tests/hlsl/ceil.shader_test +++ b/tests/hlsl/ceil.shader_test @@ -37,10 +37,7 @@ uniform 0 float4 -0.5 6.5 7.5 3.4 todo(sm<4) draw quad probe all rgba (7.0, 8.0, 0.0, 4.0) 4 -[require] -shader model >= 4.0 - -[pixel shader] +[pixel shader todo(sm<4)] uniform int4 u; float4 main() : sv_target @@ -52,6 +49,7 @@ float4 main() : sv_target } [test] -uniform 0 int4 -1 6 7 3 -draw quad +if(sm<4) uniform 0 float4 -1 6 7 3 +if(sm>=4) uniform 0 int4 -1 6 7 3 +todo(sm<4) draw quad probe all rgba (6.0, 7.0, -1.0, 3.0) 4 diff --git a/tests/hlsl/floor.shader_test b/tests/hlsl/floor.shader_test index 89e1f12ef..ed1ab7688 100644 --- a/tests/hlsl/floor.shader_test +++ b/tests/hlsl/floor.shader_test @@ -37,10 +37,8 @@ uniform 0 float4 -0.5 6.5 7.5 3.4 todo(sm<4) draw quad probe all rgba (6.0, 7.0, -1.0, 3.0) 4 -[require] -shader model >= 4.0 -[pixel shader] +[pixel shader todo(sm<4)] uniform int4 u; float4 main() : sv_target @@ -52,6 +50,7 @@ float4 main() : sv_target } [test] -uniform 0 int4 -1 6 7 3 -draw quad +if(sm<4) uniform 0 float4 -1 6 7 3 +if(sm>=4) uniform 0 int4 -1 6 7 3 +todo(sm<4) draw quad probe all rgba (6.0, 7.0, -1.0, 3.0) 4 diff --git a/tests/hlsl/function-cast.shader_test b/tests/hlsl/function-cast.shader_test index c92289863..f049b79d1 100644 --- a/tests/hlsl/function-cast.shader_test +++ b/tests/hlsl/function-cast.shader_test @@ -69,8 +69,6 @@ uniform 0 float4 -1.9 -1.0 2.9 4.0 todo draw quad probe all rgba (-1.0, -1.0, 2.0, 4.0) -[require] -shader model >= 4.0 [pixel shader todo] uniform int4 i; @@ -88,6 +86,7 @@ float4 main() : sv_target } [test] -uniform 0 int4 -2 0 1 -3000000 +if(sm<4) uniform 0 float4 -2 0 1 -3000000 +if(sm>=4) uniform 0 int4 -2 0 1 -3000000 todo(sm<6) draw quad -probe all rgba (-1.0, 0.0, 1.0, -3000000.0) +probe all rgba (-1.0, 0.0, 1.0, -3000000.0) 4 diff --git a/tests/hlsl/ldexp.shader_test b/tests/hlsl/ldexp.shader_test index f8ad40d8e..2d778e077 100644 --- a/tests/hlsl/ldexp.shader_test +++ b/tests/hlsl/ldexp.shader_test @@ -13,10 +13,8 @@ uniform 4 float4 0.0 -10.0 10.0 100.0 draw quad probe all rgba (2.0, 0.00292968750, 4096.0, 6.33825300e+030) 2 -[require] -shader model >= 4.0 -[pixel shader] +[pixel shader todo(sm<4)] uniform int4 x; uniform int4 y; @@ -26,9 +24,11 @@ float4 main() : SV_TARGET } [test] -uniform 0 int4 2 3 4 5 -uniform 4 int4 0 -10 10 100 -draw quad +if(sm<4) uniform 0 float4 2 3 4 5 +if(sm<4) uniform 4 float4 0 -10 10 100 +if(sm>=4) uniform 0 int4 2 3 4 5 +if(sm>=4) uniform 4 int4 0 -10 10 100 +todo(sm<4) draw quad probe all rgba (2.0, 0.00292968750, 4096.0, 6.33825300e+030) 2 diff --git a/tests/hlsl/lerp.shader_test b/tests/hlsl/lerp.shader_test index 15e90cef9..901857dfd 100644 --- a/tests/hlsl/lerp.shader_test +++ b/tests/hlsl/lerp.shader_test @@ -15,10 +15,8 @@ uniform 8 float4 0.0 1.0 -1.0 0.75 draw quad probe all rgba (2.0, -10.0, -2.0, 76.25) -[require] -shader model >= 4.0 -[pixel shader] +[pixel shader todo(sm<4)] uniform int4 x; uniform int4 y; uniform int4 s; @@ -29,10 +27,13 @@ float4 main() : SV_TARGET } [test] -uniform 0 int4 2 3 4 0 -uniform 4 int4 0 -10 10 1000000 -uniform 8 int4 0 1 -1 1000000 -draw quad +if(sm<4) uniform 0 float4 2 3 4 0 +if(sm<4) uniform 4 float4 0 -10 10 1000000 +if(sm<4) uniform 8 float4 0 1 -1 1000000 +if(sm>=4) uniform 0 int4 2 3 4 0 +if(sm>=4) uniform 4 int4 0 -10 10 1000000 +if(sm>=4) uniform 8 int4 0 1 -1 1000000 +todo(sm<4) draw quad probe all rgba (2.0, -10.0, -2.0, 1e12) diff --git a/tests/hlsl/sign.shader_test b/tests/hlsl/sign.shader_test index 6ec5a571d..5d1c18486 100644 --- a/tests/hlsl/sign.shader_test +++ b/tests/hlsl/sign.shader_test @@ -17,6 +17,7 @@ uniform 0 float4 0.0 0.0 0.0 0.0 todo(sm<4) draw quad probe all rgba (0.0, 0.0, 0.0, 0.0) + [pixel shader todo(sm<4)] uniform float4 f; @@ -30,6 +31,7 @@ uniform 0 float4 1.0 2.0 3.0 4.0 todo(sm<4) draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) + [pixel shader todo(sm<4)] uniform float2x2 f; @@ -44,9 +46,6 @@ uniform 4 float4 3.0 4.0 0.0 0.0 todo(sm<4) draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) -[require] -% SM1-3 doesn't support integral types -shader model >= 4.0 [pixel shader todo(sm<4)] uniform int f; @@ -57,16 +56,20 @@ float4 main() : sv_target } [test] -uniform 0 int4 1 0 0 0 -draw quad +if(sm<4) uniform 0 float4 1 0 0 0 +if(sm>=4) uniform 0 int4 1 0 0 0 +todo(sm<4) draw quad probe all rgba (1, 1, 1, 1) -uniform 0 int4 -1 0 0 0 -draw quad +if(sm<4) uniform 0 float4 -1 0 0 0 +if(sm>=4) uniform 0 int4 -1 0 0 0 +todo(sm<4) draw quad probe all rgba (-1, -1, -1, -1) -uniform 0 int4 0 0 0 0 -draw quad +if(sm<4) uniform 0 float4 0 0 0 0 +if(sm>=4) uniform 0 int4 0 0 0 0 +todo(sm<4) draw quad probe all rgba (0, 0, 0, 0) + [pixel shader todo(sm<4)] uniform int4 f; @@ -76,10 +79,12 @@ float4 main() : sv_target } [test] -uniform 0 int4 1 2 3 4 -draw quad +if(sm<4) uniform 0 float4 1 2 3 4 +if(sm>=4) uniform 0 int4 1 2 3 4 +todo(sm<4) draw quad probe all rgba (1, 1, 1, 1) + [pixel shader todo(sm<4)] uniform int2x2 f; @@ -89,7 +94,9 @@ float4 main() : sv_target } [test] -uniform 0 int4 1 2 0 0 -uniform 4 int4 3 4 0 0 -draw quad +if(sm<4) uniform 0 float4 1 2 0 0 +if(sm<4) uniform 4 float4 3 4 0 0 +if(sm>=4) uniform 0 int4 1 2 0 0 +if(sm>=4) uniform 4 int4 3 4 0 0 +todo(sm<4) draw quad probe all rgba (1, 1, 1, 1) diff --git a/tests/hlsl/switch.shader_test b/tests/hlsl/switch.shader_test index 63c9c09da..1da5d12d0 100644 --- a/tests/hlsl/switch.shader_test +++ b/tests/hlsl/switch.shader_test @@ -1,7 +1,4 @@ -[require] -shader model >= 4.0 - -[pixel shader] +[pixel shader todo(sm<4)] uint4 v; float4 main() : sv_target @@ -18,18 +15,22 @@ float4 main() : sv_target } [test] -uniform 0 uint4 3 0 0 0 -draw quad +if(sm<4) uniform 0 float4 3 0 0 0 +if(sm>=4) uniform 0 uint4 3 0 0 0 +todo(sm<4) draw quad probe all rgba (5.0, 5.0, 5.0, 5.0) -uniform 0 uint4 1 0 0 0 -draw quad +if(sm<4) uniform 0 float4 1 0 0 0 +if(sm>=4) uniform 0 uint4 1 0 0 0 +todo(sm<4) draw quad probe all rgba (4.0, 4.0, 4.0, 4.0) -uniform 0 uint4 0 0 0 0 -draw quad +if(sm<4) uniform 0 float4 0 0 0 0 +if(sm>=4) uniform 0 uint4 0 0 0 0 +todo(sm<4) draw quad probe all rgba (3.0, 3.0, 3.0, 3.0) + % just a default case -[pixel shader] +[pixel shader todo(sm<4)] uint4 v; float4 main() : sv_target @@ -42,16 +43,20 @@ float4 main() : sv_target } [test] -uniform 0 uint4 3 0 0 0 -draw quad +if(sm<4) uniform 0 float4 3 0 0 0 +if(sm>=4) uniform 0 uint4 3 0 0 0 +todo(sm<4) draw quad probe all rgba (5.0, 5.0, 5.0, 5.0) -uniform 0 uint4 1 0 0 0 -draw quad +if(sm<4) uniform 0 float4 1 0 0 0 +if(sm>=4) uniform 0 uint4 1 0 0 0 +todo(sm<4) draw quad probe all rgba (5.0, 5.0, 5.0, 5.0) -uniform 0 uint4 0 0 0 0 -draw quad +if(sm<4) uniform 0 float4 0 0 0 0 +if(sm>=4) uniform 0 uint4 0 0 0 0 +todo(sm<4) draw quad probe all rgba (5.0, 5.0, 5.0, 5.0) + % completely empty [pixel shader fail] uint4 v; @@ -63,8 +68,9 @@ float4 main() : sv_target } } + % falling through is only supported for empty case statements -[pixel shader] +[pixel shader todo(sm<4)] uint4 v; float4 main() : sv_target @@ -83,18 +89,22 @@ float4 main() : sv_target } [test] -uniform 0 uint4 2 0 0 0 -draw quad +if(sm<4) uniform 0 float4 2 0 0 0 +if(sm>=4) uniform 0 uint4 2 0 0 0 +todo(sm<4) draw quad probe all rgba (1.0, 2.0, 3.0, 4.0) -uniform 0 uint4 1 0 0 0 -draw quad +if(sm<4) uniform 0 float4 1 0 0 0 +if(sm>=4) uniform 0 uint4 1 0 0 0 +todo(sm<4) draw quad probe all rgba (1.1, 2.0, 3.0, 4.0) -uniform 0 uint4 0 0 0 0 -draw quad +if(sm<4) uniform 0 float4 0 0 0 0 +if(sm>=4) uniform 0 uint4 0 0 0 0 +todo(sm<4) draw quad probe all rgba (1.1, 2.0, 3.0, 4.0) + % case value evaluation -[pixel shader] +[pixel shader todo(sm<4)] uint4 v; float4 main() : sv_target @@ -115,15 +125,18 @@ float4 main() : sv_target } [test] -uniform 0 uint4 2 0 0 0 -draw quad +if(sm<4) uniform 0 float4 2 0 0 0 +if(sm>=4) uniform 0 uint4 2 0 0 0 +todo(sm<4) draw quad probe all rgba (1.1, 2.1, 3.1, 4.1) -uniform 0 uint4 1 0 0 0 -draw quad +if(sm<4) uniform 0 float4 1 0 0 0 +if(sm>=4) uniform 0 uint4 1 0 0 0 +todo(sm<4) draw quad probe all rgba (1.0, 2.0, 3.0, 4.0) + % floats are accepted -[pixel shader fail(sm>=6)] +[pixel shader fail(sm>=6) todo(sm<4)] uint4 v; float4 main() : sv_target @@ -144,14 +157,16 @@ float4 main() : sv_target } [test] -uniform 0 uint4 2 0 0 0 -draw quad +if(sm<4) uniform 0 float4 2 0 0 0 +if(sm>=4) uniform 0 uint4 2 0 0 0 +todo(sm<4) draw quad probe all rgba (1.1, 2.1, 3.1, 4.1) -uniform 0 uint4 1 0 0 0 -draw quad +if(sm<4) uniform 0 float4 1 0 0 0 +if(sm>=4) uniform 0 uint4 1 0 0 0 +todo(sm<4) draw quad probe all rgba (1.0, 2.0, 3.0, 4.0) -[pixel shader fail(sm>=6)] +[pixel shader fail(sm<4 | sm>=6) todo(sm<4)] float4 v; float4 main() : sv_target @@ -173,10 +188,10 @@ float4 main() : sv_target [test] uniform 0 float4 2.0 0.0 0.0 0.0 -draw quad +todo(sm<4) draw quad probe all rgba (1.1, 2.1, 3.1, 4.1) uniform 0 float4 1.0 0.0 0.0 0.0 -draw quad +todo(sm<4) draw quad probe all rgba (1.0, 2.0, 3.0, 4.0) [pixel shader fail] @@ -347,7 +362,7 @@ float4 main() : sv_target } % more complicated breaks -[pixel shader] +[pixel shader todo(sm<4)] uint4 v; float4 main() : sv_target @@ -373,18 +388,21 @@ float4 main() : sv_target } [test] -uniform 0 uint4 2 0 0 0 -draw quad +if(sm<4) uniform 0 float4 2 0 0 0 +if(sm>=4) uniform 0 uint4 2 0 0 0 +todo(sm<4) draw quad probe all rgba (1.1, 2.1, 3.1, 4.1) -uniform 0 uint4 1 0 0 0 -draw quad +if(sm<4) uniform 0 float4 1 0 0 0 +if(sm>=4) uniform 0 uint4 1 0 0 0 +todo(sm<4) draw quad probe all rgba (1.2, 2.2, 3.2, 4.2) -uniform 0 uint4 0 0 0 0 -draw quad +if(sm<4) uniform 0 float4 0 0 0 0 +if(sm>=4) uniform 0 uint4 0 0 0 0 +todo(sm<4) draw quad probe all rgba (1.0, 2.0, 3.0, 4.0) % switch breaks within a loop -[pixel shader] +[pixel shader todo(sm<4)] uint4 v; float4 main() : sv_target @@ -411,12 +429,14 @@ float4 main() : sv_target } [test] -uniform 0 uint4 2 0 0 0 -draw quad +if(sm<4) uniform 0 float4 2 0 0 0 +if(sm>=4) uniform 0 uint4 2 0 0 0 +todo(sm<4) draw quad probe all rgba (5.0, 6.0, 7.0, 8.0) + % default case placement -[pixel shader] +[pixel shader todo(sm<4)] uint4 v; float4 main() : sv_target @@ -442,17 +462,21 @@ float4 main() : sv_target } [test] -uniform 0 uint4 0 0 0 0 -draw quad +if(sm<4) uniform 0 float4 0 0 0 0 +if(sm>=4) uniform 0 uint4 0 0 0 0 +todo(sm<4) draw quad probe all rgba (4.0, 5.0, 6.0, 7.0) -uniform 0 uint4 2 0 0 0 -draw quad +if(sm<4) uniform 0 float4 2 0 0 0 +if(sm>=4) uniform 0 uint4 2 0 0 0 +todo(sm<4) draw quad probe all rgba (2.0, 3.0, 4.0, 5.0) -uniform 0 uint4 3 0 0 0 -draw quad +if(sm<4) uniform 0 float4 3 0 0 0 +if(sm>=4) uniform 0 uint4 3 0 0 0 +todo(sm<4) draw quad probe all rgba (4.0, 5.0, 6.0, 7.0) -[pixel shader] + +[pixel shader todo(sm<4)] uint4 v; float4 main() : sv_target @@ -479,14 +503,17 @@ float4 main() : sv_target } [test] -uniform 0 uint4 3 0 0 0 -draw quad +if(sm<4) uniform 0 float4 3 0 0 0 +if(sm>=4) uniform 0 uint4 3 0 0 0 +todo(sm<4) draw quad probe all rgba (1.0, 2.0, 3.0, 4.0) -uniform 0 uint4 0 0 0 0 -draw quad +if(sm<4) uniform 0 float4 0 0 0 0 +if(sm>=4) uniform 0 uint4 0 0 0 0 +todo(sm<4) draw quad probe all rgba (4.0, 5.0, 6.0, 7.0) -uniform 0 uint4 5 0 0 0 -draw quad +if(sm<4) uniform 0 float4 5 0 0 0 +if(sm>=4) uniform 0 uint4 5 0 0 0 +todo(sm<4) draw quad probe all rgba (1.0, 2.0, 3.0, 4.0) % 'continue' is not supported in switches @@ -515,7 +542,7 @@ float4 main() : sv_target return c; } -[pixel shader] +[pixel shader fail(sm<4) todo(sm<4)] uint4 v; float4 main() : sv_target @@ -545,15 +572,17 @@ float4 main() : sv_target } [test] -uniform 0 uint4 1 0 3 1 +if(sm<4) uniform 0 float4 0 0 3 1 +if(sm>=4) uniform 0 uint4 0 0 3 1 draw quad -probe all rgba (10.0, 11.0, 12.0, 13.0) -uniform 0 uint4 0 0 3 1 +todo(sm>=4) probe all rgba (10.0, 11.0, 12.0, 13.0) +if(sm<4) uniform 0 float4 1 0 3 1 +if(sm>=4) uniform 0 uint4 1 0 3 1 draw quad -probe all rgba (7.0, 8.0, 9.0, 10.0) +todo(sm>=4) probe all rgba (7.0, 8.0, 9.0, 10.0) % return from a switch nested in a loop -[pixel shader] +[pixel shader fail(sm<4) todo(sm<4)] uint4 v; float4 main() : sv_target @@ -579,9 +608,11 @@ float4 main() : sv_target } [test] -uniform 0 uint4 0 0 3 1 +if(sm<4) uniform 0 float4 0 0 3 1 +if(sm>=4) uniform 0 uint4 0 0 3 1 draw quad probe all rgba (304.0, 305.0, 306.0, 307.0) -uniform 0 uint4 1 0 3 1 +if(sm<4) uniform 0 float4 1 0 3 1 +if(sm>=4) uniform 0 uint4 1 0 3 1 draw quad probe all rgba (3.0, 4.0, 5.0, 6.0) diff --git a/tests/hlsl/trunc.shader_test b/tests/hlsl/trunc.shader_test index f1d23bf82..d57438641 100644 --- a/tests/hlsl/trunc.shader_test +++ b/tests/hlsl/trunc.shader_test @@ -30,10 +30,8 @@ uniform 0 float4 -0.5 6.5 7.5 3.4 todo(sm<4) draw quad probe all rgba (6.0, 7.0, 0.0, 3.0) -[require] -shader model >= 4.0 -[pixel shader] +[pixel shader todo(sm<4)] uniform int4 u; float4 main() : sv_target @@ -45,6 +43,7 @@ float4 main() : sv_target } [test] -uniform 0 int4 -1 6 7 3 -draw quad +if(sm<4) uniform 0 float4 -1 6 7 3 +if(sm>=4) uniform 0 int4 -1 6 7 3 +todo(sm<4) draw quad probe all rgba (6.0, 7.0, -1.0, 3.0) -- GitLab https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/616
```diff @@ -545,15 +572,17 @@ float4 main() : sv_target }
[test] -uniform 0 uint4 1 0 3 1 +if(sm<4) uniform 0 float4 0 0 3 1 +if(sm>=4) uniform 0 uint4 0 0 3 1 draw quad -probe all rgba (10.0, 11.0, 12.0, 13.0) -uniform 0 uint4 0 0 3 1 +todo(sm>=4) probe all rgba (10.0, 11.0, 12.0, 13.0) +if(sm<4) uniform 0 float4 1 0 3 1 +if(sm>=4) uniform 0 uint4 1 0 3 1 ```
That looks like a rebase glitch after 4ba324d37c8fa7ca47525f2525ded391a0b851cc. -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/616#note_60904
participants (3)
-
Francisco Casas -
Francisco Casas (＠fcasas)
-
Henri Verbeet (＠hverbeet)