Skip to content

[SM6.10] Retail hash SM6.10, enable experimental SM6.11 - #8957

Merged
Ashley Coleman (V-FEXrt) merged 4 commits into
microsoft:mainfrom
V-FEXrt:sm610-retail-sm611-experimental
Sep 24, 2026
Merged

Ashley Coleman (V-FEXrt) merged 4 commits into
microsoft:mainfrom
V-FEXrt:sm610-retail-sm611-experimental

Conversation

@V-FEXrt

Copy link
Copy Markdown
Collaborator
  • Retail hash SM6.10 shaders
  • Enable the spelling of SM6.11 shaders
  • Update tests

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Fixed-version metadata remains at 1.9.2609.0 while release metadata advances to 1.10.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Retails Shader Model 6.10/DXIL 1.10 and enables experimental Shader Model 6.11 profiles.

Changes:

  • Advances released and experimental DXIL/shader-model versions.
  • Adds SM 6.11 profiles and updates generated metadata.
  • Updates validation, reflection tests, and release notes.
File Description
utils/​version/​latest-release.json Advances release metadata to 1.10.
utils/​hct/​hctdb_instrhelp.py Sets SM 6.11 as highest recognized version.
tools/​clang/​unittests/​HLSL/​ValidationTest.cpp Tests retail hashing on the latest release.
tools/​clang/​test/​LitDXILValidation/​experimental-op-on-release-sm.ll Moves released-model validation coverage to 6.10.
tools/​clang/​test/​DXC/​experimental-dxil-6-11-op.ll Tests experimental SM 6.11 validation and preview hashing.
tools/​clang/​test/​CodeGenDXIL/​reflection/​experimental_rdat.hlsl Updates experimental RDAT coverage to 6.11.
tools/​clang/​test/​CodeGenDXIL/​reflection/​experimental_rdat_rel_sm.hlsl Covers released 6.10 and experimental 6.11 RDAT.
lib/​DXIL/​DxilShaderModel.cpp Registers SM 6.11 profiles and version mappings.
include/​dxc/​Support/​HLSLOptions.td Lists SM 6.11 target profiles.
include/​dxc/​DXIL/​DxilShaderModel.h Updates model limits and profile count.
include/​dxc/​DXIL/​DxilConstants.h Advances DXIL recognized/released versions.
docs/​ReleaseNotes.md Documents retail 6.10 and experimental 6.11.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread utils/version/latest-release.json
Copilot AI review requested due to automatic review settings September 23, 2026 22:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Fixed-version builds still report version 1.9, and the new release-note entry needs correction.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 Medium severity · 1 Low severity

Open (2)

Comment thread docs/ReleaseNotes.md Outdated
Copilot AI review requested due to automatic review settings September 23, 2026 22:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The implementation and tests are consistent, with only minor release-note wording cleanup identified.

Review effort: Balanced
Findings: None

Resolved since last review (2)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The compiler-wide release-boundary change requires final human validation of release metadata and validator compatibility.

Review effort: Balanced
Findings: None

Comment thread docs/ReleaseNotes.md

#### HLSL Language

- Starting with HLSL 202x, the count in `[unroll(N)]` is a partial-unroll hint

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I see this is just moving existing release notes but mixing 202x and 2026 is confusing since I thought those were the same?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

microsoft/hlsl-specs#931 They are being split :)

Comment thread tools/clang/test/DXC/experimental-dxil-6-11-op.ll

@llvm-beanz Chris B (llvm-beanz) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM.

We should consider removing geometry shaders in SM 6.11, but we should shuttle that through the proposal process before we make changes based on it.

@V-FEXrt
Ashley Coleman (V-FEXrt) merged commit d02056d into microsoft:main Sep 24, 2026
12 checks passed
@V-FEXrt
Ashley Coleman (V-FEXrt) deleted the sm610-retail-sm611-experimental branch September 24, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants