Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't include d3dcompiler_47.dll on ARM64. #6496

Merged
merged 1 commit into from
May 9, 2022

Conversation

singhashish-wpf
Copy link
Member

@singhashish-wpf singhashish-wpf commented Apr 29, 2022

Fixes Issue #5462
Main PR #5747

Description

Don't include d3dcompiler_47.dll on ARM64.
The included ARM64 version of d3dcompiler_47 isn’t real – it’s a stub that looks and smells real, but as soon as you try to use it, the app explodes. So, removing it for arm64.

Customer Impact

Without this change the d3dcompiler.dll which is shipped for arm64 is causing issues with 3rd party libraries.
In some cases when starting the application, the chromium browser hangs, and the log file contains an error that it cannot find the D3DCompile entry point

Regression

No

Testing

Build and CI

Risk

None.

@singhashish-wpf singhashish-wpf requested a review from a team as a code owner April 29, 2022 04:37
@ghost ghost assigned singhashish-wpf Apr 29, 2022
@ghost ghost added the PR metadata: Label to tag PRs, to facilitate with triage label Apr 29, 2022
@ghost ghost requested review from dipeshmsft and SamBent April 29, 2022 04:37
@ghost
Copy link

ghost commented Apr 29, 2022

Hi @singhashish-wpf. Please make sure you've updated the PR description to use the Shiproom Template. Also, make sure this PR is not marked as a draft and is ready-to-merge.

To learn more about how to prepare a servicing PR click here.

@singhashish-wpf singhashish-wpf merged commit 46cc7ff into release/6.0 May 9, 2022
@singhashish-wpf singhashish-wpf deleted the d3dcompiler6 branch May 9, 2022 03:49
@ghost ghost locked as resolved and limited conversation to collaborators Jun 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR metadata: Label to tag PRs, to facilitate with triage Servicing-approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants