Skip to content

Exec test - WAR SM feature check on older runtime#3603

Merged
pow2clk merged 1 commit intomicrosoft:masterfrom
pow2clk:exectest_sm_check
Mar 18, 2021
Merged

Exec test - WAR SM feature check on older runtime#3603
pow2clk merged 1 commit intomicrosoft:masterfrom
pow2clk:exectest_sm_check

Conversation

@pow2clk
Copy link
Copy Markdown
Collaborator

@pow2clk pow2clk commented Mar 18, 2021

An earlier version of the D3D runtime doesn't handle CheckFeatureSupport
for FEATURE_SHADER_MODEL when the corresponding struct is set to a
unrecognized shader model. Instead of returning the highest supported
shader model that is less than the one provided as documented, the call
fails. In practice, this only occurred for 6.6 tests where the SDK had
6.6 capability, but the installed runtime did not.

To work around this, we don't verify that the call succeeds. Instead,
a failing return result is interpretted the same way as when the highest
shader model value returned is lower than that requested.

An earlier version of the D3D runtime doesn't handle CheckFeatureSupport
for FEATURE_SHADER_MODEL when the corresponding struct is set to a
unrecognized shader model. Instead of returning the highest supported
shader model that is less than the one provided as documented, the call
fails. In practice, this only occurred for 6.6 tests where the SDK had
6.6 capability, but the installed runtime did not.

To work around this, we don't verify that the call succeeds. Instead,
a failing return result is interpretted the same way as a highest shader
model value that is lower than that requested.
@pow2clk pow2clk requested review from hekota, jenatali and tex3d March 18, 2021 09:51
@AppVeyorBot
Copy link
Copy Markdown

@pow2clk pow2clk force-pushed the exectest_sm_check branch from 6d03a5d to 13a1320 Compare March 18, 2021 13:26
@AppVeyorBot
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

@hekota hekota left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@pow2clk pow2clk merged commit ddde121 into microsoft:master Mar 18, 2021
@pow2clk pow2clk deleted the exectest_sm_check branch March 18, 2021 18:03
pow2clk pushed a commit to pow2clk/DirectXShaderCompiler that referenced this pull request Mar 18, 2021
Merge  microsoft#3603 into release-1.6.2104

An earlier version of the D3D runtime doesn't handle CheckFeatureSupport
for FEATURE_SHADER_MODEL when the corresponding struct is set to a
unrecognized shader model. Instead of returning the highest supported
shader model that is less than the one provided as documented, the call
fails. In practice, this only occurred for 6.6 tests where the SDK had
6.6 capability, but the installed runtime did not.

To work around this, we don't verify that the call succeeds. Instead,
a failing return result is interpretted the same way as a highest shader
model value that is lower than that requested.

(cherry picked from commit ddde121)
pow2clk pushed a commit that referenced this pull request Mar 18, 2021
Merge  #3603 into release-1.6.2104

An earlier version of the D3D runtime doesn't handle CheckFeatureSupport
for FEATURE_SHADER_MODEL when the corresponding struct is set to a
unrecognized shader model. Instead of returning the highest supported
shader model that is less than the one provided as documented, the call
fails. In practice, this only occurred for 6.6 tests where the SDK had
6.6 capability, but the installed runtime did not.

To work around this, we don't verify that the call succeeds. Instead,
a failing return result is interpretted the same way as a highest shader
model value that is lower than that requested.

(cherry picked from commit ddde121)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants