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
ILDASM /linenum support? #53822
Comments
@EgorBo PTAL. |
When ilasm received the support for portable PDB, ildasm was left out #37702 (comment). It is more of a feature request than a bug report. 😄 Note that once portable PDB parser is plugged into ildasm, this condition will start to satisfy: runtime/src/coreclr/ildasm/dis.cpp Line 1132 in abb7df5
fInsertSourceLines = g_fInsertSourceLines; under if (g_pSymReader) condition, g_fInsertSourceLines is 1 with /linenum or -linenum argument, otherwise 0; but that statement is unreachable because g_pSymReader is always null in ildasm.
|
@am11 Thank you for the added context here, it's very helpful. We should definitely support portable PDBs for this; it would help make F# debug testing a bit easier. |
Hello, I decided to work on this (as I worked on the ilasm support for portable PDB as well :)) as part of the hackathon project. |
@ivanpovazan: Is there any progress on this issue? We use ILASM/ILDASM roundtrips for an assembly enhancer with C# (see www.netdataobjects.de). ILDASM doesn't generate .line statements with .Net Standard DLLs. That's really bad, because users can't debug enhanced DLLs. |
@mirkomaty I made some initial progress like reading the document table, but had to switch to some other higher priority issues. |
@ivanpovazan: Thanks a lot for your answer. The problem is not so urgent, because currently I tell my users to use
But since .NET Core will be the dominant platform in the future, there is little point in having my users insert that line into their project files all the time. I'm not able to customize ILDasm myself, so I appreciate you taking the time to do so. ;-) If it happens this year, it is perfect for me. Thanks again. |
Unfortunately, I will not have enough cycles to look further into this in the near future. Will remain at disposal if anyone needs help or has questions around this. |
The is either a question or a bug report. Basically does the
/linenum
flag of ILDASM work with .NET Core?Context: We use ILDASM in the F# repo from package “Microsoft.NETCore.ILDAsm". However I can’t get the “/linenum” option to workm, e.g. to show the PDB file contents in terms of sequence points. We use this to lock down our debugging experience, which is quite important one way or another. I could dump the sequence points from the compiler but it’s really convenient to have them showing with the IL code.
Repro:
ildasm /linenum /text foo.dll
e.g. in F# repo:
Expected: output include
.line
Actual: no
.line
observedWe've tried lots of variations like embedded symbols etc., we can't find a configuration that works.
cc @vzarytovskii, @brettfo
The text was updated successfully, but these errors were encountered: