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.

Sign up for GitHub

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

Jump to bottom

Provide symbols for end user debugging. #5562

Open
moonheart08 opened this issue Nov 14, 2024 · 1 comment · May be fixed by #5665
Open

Provide symbols for end user debugging. #5562

moonheart08 opened this issue Nov 14, 2024 · 1 comment · May be fixed by #5665
Assignees
@expipiplus1
Labels
goal:client support Feature or fix needed for a current slang user.
Milestone
Q4 2024 (Fall)

Comments

@moonheart08
Copy link

I'm currently trying to prepare reproductions for a few issues I ran into attempting to write C# bindings for the slang compiler (including what appears to be incorrect ABI for COM interfaces.), and at this time it looks like in order to get proper stack traces I would need to build all of slang locally. (Something that is beyond the scope of my current project.)

Release builds of slang should at least include a .pdb file so windbg and other debuggers can provide proper stack traces. All other DLLs that are shipped as part of the Vulkan SDK include symbols and the lack of them for slang has proven problematic for debugging FFI/interop issues.
@csyonghe
Copy link
Collaborator

@expipiplus1 It looks like although we are using relWithDebInfo config in our CI process, we are not packaging up the pdb files in the final package. Can you patch this?

@csyonghe csyonghe added the goal:client support Feature or fix needed for a current slang user. label Nov 14, 2024
@csyonghe csyonghe added this to the Q4 2024 (Fall) milestone Nov 14, 2024
@moonheart08 moonheart08 mentioned this issue Nov 15, 2024
Closed
expipiplus1 added a commit to expipiplus1/slang that referenced this issue Nov 25, 2024
@expipiplus1
Install PDB files when available
76519c4 
Closes shader-slang#5562
@expipiplus1 expipiplus1 linked a pull request Nov 25, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@expipiplus1 expipiplus1

Labels
goal:client support Feature or fix needed for a current slang user.
Projects
None yet
Milestone
Q4 2024 (Fall)
Development

Successfully merging a pull request may close this issue.

Install PDB files when available expipiplus1/slang
3 participants
@expipiplus1 @csyonghe @moonheart08