Skip to content

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

Merged
pow2clk merged 1 commit intomicrosoft:release-1.6.2104from
pow2clk:release_merge
Mar 18, 2021
Merged

Exec test - WAR SM feature check on older runtime#3605
pow2clk merged 1 commit intomicrosoft:release-1.6.2104from
pow2clk:release_merge

Conversation

@pow2clk
Copy link
Copy Markdown
Collaborator

@pow2clk pow2clk commented 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)

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 pow2clk requested a review from hekota March 18, 2021 18:27
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 6b8d715 into microsoft:release-1.6.2104 Mar 18, 2021
@pow2clk pow2clk deleted the release_merge branch March 18, 2021 19:03
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.

2 participants