Skip to content
/ cecil Public
forked from jbevain/cecil
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

Preserve Documents debug info #270

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sbomer
Copy link
Member

@sbomer sbomer commented Feb 18, 2025

Currently when writing PDBs, Document debug info is only preserved when referenced from a sequence point. It should be preserved always, because Documents don't necessarily correspond to any sequence points (for example, C# code that contains an interface definition with no implementation).

This will preserve the Document debug info always.

Fixes dotnet/runtime#112049
Fixes dotnet/runtime#111138

@sbomer sbomer requested a review from a team February 18, 2025 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Trimmed facade assemblies fail binskim SourceLink returning incorrect PDB info
1 participant