Henri Verbeet pushed to branch master at wine / vkd3d
Commits:
c6816739 by Henri Verbeet at 2024-12-17T16:42:53+01:00
vkd3d-shader/d3dbc: Separate writing the comment bytecode from write_sm1_uniforms().
Consistent with how D3DXFindShaderComment() allows looking up comments
by tag. This also makes it a little easier to move CTAB generation out
of d3dbc.c.
- - - - -
e8cc2884 by Henri Verbeet at 2024-12-17T16:42:53+01:00
vkd3d-shader/hlsl: Move CTAB generation to hlsl_codegen.c.
- - - - -
3 changed files:
- libs/vkd3d-shader/d3dbc.c
- libs/vkd3d-shader/hlsl.h
- libs/vkd3d-shader/hlsl_codegen.c
View it on GitLab: https://gitlab.winehq.org/wine/vkd3d/-/compare/00538c377e58317a2a5f625a9b0c…
--
View it on GitLab: https://gitlab.winehq.org/wine/vkd3d/-/compare/00538c377e58317a2a5f625a9b0c…
You're receiving this email because of your account on gitlab.winehq.org.
Henri Verbeet pushed to branch master at wine / vkd3d
Commits:
8d4b790e by Giovanni Mascellani at 2024-12-17T16:39:32+01:00
vkd3d-shader/ir: Represent information about I/O registers with a table.
It makes the code quite longer, but also easier to read and extend
with further properties.
A (desirable) side effect of this commit is that it is checked
whether I/O register types are legal depending on the shader type
and phase, while before that was assumed.
This commit introduces enum vsir_io_reg_type and enum vsir_phase
which shadow enum vkd3d_shader_register_type and enum
vkd3d_shader_opcode, with the goal of making the data tables
smaller.
- - - - -
38a5c905 by Giovanni Mascellani at 2024-12-17T16:39:32+01:00
vkd3d-shader/ir: Validate INPUT destination parameters using a uniform helper.
- - - - -
b194e5dc by Giovanni Mascellani at 2024-12-17T16:39:32+01:00
vkd3d-shader/ir: Validate PATCHCONST destination parameters using a uniform helper.
- - - - -
7d87b4e8 by Giovanni Mascellani at 2024-12-17T16:39:32+01:00
vkd3d-shader/ir: Validate OUTPUT destination parameters.
- - - - -
3dc7f322 by Giovanni Mascellani at 2024-12-17T16:39:32+01:00
vkd3d-shader/ir: Validate INCONTROLPOINT destination parameters.
- - - - -
00538c37 by Giovanni Mascellani at 2024-12-17T16:39:32+01:00
vkd3d-shader/ir: Validate OUTCONTROLPOINT destination parameters.
- - - - -
1 changed file:
- libs/vkd3d-shader/ir.c
View it on GitLab: https://gitlab.winehq.org/wine/vkd3d/-/compare/27c77769478cd68e5ffcd143fbc5…
--
View it on GitLab: https://gitlab.winehq.org/wine/vkd3d/-/compare/27c77769478cd68e5ffcd143fbc5…
You're receiving this email because of your account on gitlab.winehq.org.
Henri Verbeet pushed to branch master at wine / vkd3d
Commits:
e91c07e1 by Elizabeth Figura at 2024-12-17T16:18:21+01:00
tests/shader_runner: Record HLSL todo/fail state for each shader model.
When a shader fails to compile for a range of versions, we want to validate that
we are correctly implementing that behaviour. E.g. if a feature requires shader
model 5.0, we should validate that it compiles correctly with 5.0 (which we do),
but also that it *fails* to compile with 4.1 (which we do not).
The obvious and simple solution is to simply run compile tests for each version.
There are, however, at least 12 versions of HLSL up to and including 6.0, at
least 10 of which are known to introduce new features. Shader compilation takes
about 10-15% of the time that draw and dispatch does, both for native and
(currently) vkd3d. Testing every version for every shader would add a
noticeable amount of time to the tests.
In practice, the interesting versions to test for most shaders are:
* At least one from each range 1-3, 4-5, and 6. It's common enough for the
semantics of the HLSL to differ between bytecode formats, or for features to
be added or removed across those boundaries.
* If the shader requires a given feature, we want to test both sides of the cusp
to ensure we're requiring the same version for the feature.
In practice this is 3 or 4 versions, which is measurably less than the 12 we'd
otherwise be running.
In order to achieve this goal of testing only the 3 or 4 interesting versions
for a shader, we need to know what version is actually required for a feature.
This is encoded in the shader itself using e.g. [pixel shader fail(sm<5)].
This patch therefore implements the first step towards this goal, namely,
determining which versions succeed and fail, so we can figure out which ones are
interesting.
We could require the test writer to specify which versions are interesting ahead
of time (e.g. "for version in 2.0 4.1 5.0 6.0") but this is both redundant (and
there are a *lot* of tests that need some feature gate) and easy for a test
writer to get wrong by missing interesting versions.
- - - - -
067e6dee by Elizabeth Figura at 2024-12-17T16:24:00+01:00
tests/shader_runner: Test HLSL compilation in a separate pass.
Mainly in order to not waste time compile-testing the same version
more than once [as we do with e.g. the d3d11 and d3d12 runners, or
d3d12, GL, and Vulkan].
- - - - -
8fcbbfb8 by Elizabeth Figura at 2024-12-17T16:35:09+01:00
tests/shader_runner: Test versions where the compilation result changes.
Adjust the algorithm for deciding for which profiles to test compilation.
We first ensure that if the compilation result changes (most often as the result
of a feature introduced in a specific version), we test the versions immediately
on either side of the change, to validate that vkd3d-shader is emulating the
same version behaviour.
We then ensure that we are testing at least one version from each set of sm1,
sm4, and sm6.
- - - - -
27c77769 by Elizabeth Figura at 2024-12-17T16:37:41+01:00
tests: Use fail(sm<5) instead of [require] for uav-atomics.
- - - - -
14 changed files:
- tests/hlsl/cbuffer.shader_test
- tests/hlsl/object-parameters.shader_test
- tests/hlsl/object-references.shader_test
- tests/hlsl/register-reservations-profile.shader_test
- tests/hlsl/register-reservations-resources.shader_test
- tests/hlsl/sample-bias.shader_test
- tests/hlsl/sample-grad.shader_test
- tests/hlsl/sample-level.shader_test
- tests/hlsl/sampler-state.shader_test
- tests/hlsl/sampler.shader_test
- tests/hlsl/state-block-syntax.shader_test
- tests/hlsl/uav-atomics.shader_test
- tests/shader_runner.c
- tests/shader_runner.h
View it on GitLab: https://gitlab.winehq.org/wine/vkd3d/-/compare/de615609dc30cb8571e796bcd2a9…
--
View it on GitLab: https://gitlab.winehq.org/wine/vkd3d/-/compare/de615609dc30cb8571e796bcd2a9…
You're receiving this email because of your account on gitlab.winehq.org.
Henri Verbeet pushed to branch master at wine / vkd3d
Commits:
1e430886 by Elizabeth Figura at 2024-12-16T17:11:57+01:00
vkd3d-shader/d3dbc: Use struct vkd3d_shader_dst_param instead of struct sm1_dst_register.
- - - - -
77ac13e6 by Elizabeth Figura at 2024-12-16T17:11:57+01:00
vkd3d-shader/d3dbc: Use struct vkd3d_shader_src_param instead of struct sm1_src_register.
- - - - -
de615609 by Elizabeth Figura at 2024-12-16T17:11:57+01:00
vkd3d-shader/d3dbc: Use struct vkd3d_shader_instruction instead of struct sm1_instruction.
- - - - -
1 changed file:
- libs/vkd3d-shader/d3dbc.c
View it on GitLab: https://gitlab.winehq.org/wine/vkd3d/-/compare/c4f69f4d3db50710079c924f4376…
--
View it on GitLab: https://gitlab.winehq.org/wine/vkd3d/-/compare/c4f69f4d3db50710079c924f4376…
You're receiving this email because of your account on gitlab.winehq.org.