Skip to content

Comments

[HLSL] fix case-sensitive semantics for workgraphs#8029

Merged
Keenuts merged 3 commits intomicrosoft:mainfrom
Keenuts:fix-8014
Feb 18, 2026
Merged

[HLSL] fix case-sensitive semantics for workgraphs#8029
Keenuts merged 3 commits intomicrosoft:mainfrom
Keenuts:fix-8014

Conversation

@Keenuts
Copy link
Collaborator

@Keenuts Keenuts commented Jan 5, 2026

The function handling semantics for work graphs were case sensitive, but shouldn't. This commit addresses that.

Fixes #8014

Copy link
Collaborator

@s-perron s-perron left a comment

Choose a reason for hiding this comment

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

Add tests

@s-perron s-perron requested a review from bob80905 January 5, 2026 13:35
@Keenuts Keenuts requested a review from s-perron January 5, 2026 15:11
Copy link
Collaborator

@s-perron s-perron left a comment

Choose a reason for hiding this comment

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

Fine with me. We should double check with someone from MS as this affects them too.

@damyanp
Copy link
Member

damyanp commented Jan 12, 2026

If this is worth mentioning in the release notes, please add something to https://github.com/microsoft/DirectXShaderCompiler/blob/main/docs/ReleaseNotes.md as appropriate.

@Keenuts Keenuts requested a review from damyanp January 20, 2026 13:45
@Keenuts
Copy link
Collaborator Author

Keenuts commented Jan 26, 2026

@damyanp is this OK for MS?

@damyanp
Copy link
Member

damyanp commented Jan 26, 2026

@amarpMSFT / @tex3d - do you have any concerns with this change?

@amarpMSFT
Copy link
Collaborator

@amarpMSFT / @tex3d - do you have any concerns with this change?

Seems fine to me assuming this brings consistency with other semantics.

The function handling semantics for work graphs were case sensitive,
but shouldn't. This commit addresses that.

Fixes microsoft#8014
@Keenuts
Copy link
Collaborator Author

Keenuts commented Feb 17, 2026

rebased on main to fix the conflict

@Keenuts Keenuts enabled auto-merge (squash) February 17, 2026 17:07
@Keenuts Keenuts merged commit e661673 into microsoft:main Feb 18, 2026
12 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in HLSL Roadmap Feb 18, 2026
@Keenuts Keenuts deleted the fix-8014 branch February 18, 2026 14:51
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.

Work Graphs Coalescing Mode Semantics Validation is Case-Sensitive

5 participants