Skip to content

Don't pass through to DXIL.dll in DxcCreateInstance for CLSID_DxcValidator#7409

Merged
tex3d merged 1 commit intomicrosoft:mainfrom
tex3d:stop-validator-passthrough
May 1, 2025
Merged

Don't pass through to DXIL.dll in DxcCreateInstance for CLSID_DxcValidator#7409
tex3d merged 1 commit intomicrosoft:mainfrom
tex3d:stop-validator-passthrough

Conversation

@tex3d
Copy link
Copy Markdown
Contributor

@tex3d tex3d commented May 1, 2025

This change removes the code meant to pass through to DXIL.dll when creating a CLSID_DxcValidator object with DxcCreateInstance.

Since the internal validator is now the default, and it will sign shaders, there is no reason to pass through to DXIL.dll when using DxcCreateInstance on DxCompiler.dll.

Testing will come with planned work to include the ability to globally override the default validator to external.

…dator

This change removes the code meant to pass through to DXIL.dll when creating
a CLSID_DxcValidator object with DxcCreateInstance.

Since the internal validator is now the default, and it will sign shaders,
there is no reason to pass through to DXIL.dll when using DxcCreateInstance
on DxCompiler.dll.
@damyanp
Copy link
Copy Markdown
Member

damyanp commented May 1, 2025

#7397

@bob80905
Copy link
Copy Markdown
Collaborator

bob80905 commented May 1, 2025

Thanks Tex!!

@tex3d tex3d merged commit 1a75a30 into microsoft:main May 1, 2025
13 checks passed
@github-project-automation github-project-automation Bot moved this from New to Done in HLSL Roadmap May 1, 2025
@tex3d tex3d deleted the stop-validator-passthrough branch May 1, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants