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

Fix windows pdb #9229

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

garazdawi
Copy link
Contributor

This PR fixes some issues on Windows, mainly it makes sure that the correct pdb files are released so that they are part of the installer and can be used to debug crashes in erts.

@garazdawi garazdawi added team:VM Assigned to OTP team VM fix labels Dec 20, 2024
@garazdawi garazdawi added this to the OTP-28.0 milestone Dec 20, 2024
@garazdawi garazdawi requested a review from dgud December 20, 2024 12:15
@garazdawi garazdawi self-assigned this Dec 20, 2024
Visual Studio 17 seems to compile the original code
into something that make coverage into MAX_SMALL
instead of coverage_data. Extracting the data into
a variable seems to fix the error, don't know why
but hopefully it will be fixed in the compiler someday.
Or maybe the original code is somesort of undefined
behaviour I'm unable to see.

The original code compiles as it should in Visual Studio 14.
When debugging on windows, the name of the pdb file is
not the name of the file being debugger, but the name
of the file as it was compiled. We rename beam.jit.dll
to beam.smp.dll during the build so the pdb needed is called
jit.pdb and not smp.pdb.

We also make sure to release all pdb files so that they
are part of the installer.
@garazdawi garazdawi force-pushed the lukas/erts/fix-windows-pdb branch from 068fb0d to 59dd2b3 Compare December 20, 2024 12:16
Copy link
Contributor

github-actions bot commented Dec 20, 2024

CT Test Results

    3 files    141 suites   49m 2s ⏱️
1 598 tests 1 549 ✅ 49 💤 0 ❌
2 307 runs  2 238 ✅ 69 💤 0 ❌

Results for commit 0480095.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@garazdawi garazdawi force-pushed the lukas/erts/fix-windows-pdb branch from 59dd2b3 to 0505bd7 Compare December 20, 2024 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix team:VM Assigned to OTP team VM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant