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

Consolidate ilasm/main.cpp and ildasm/windasm.cpp #76773

Open
am11 opened this issue Oct 8, 2022 · 1 comment
Open

Consolidate ilasm/main.cpp and ildasm/windasm.cpp #76773

am11 opened this issue Oct 8, 2022 · 1 comment
Assignees
@AaronRobinsonMSFT
Labels
area-ILTools-coreclr Priority:3 Work that is nice to have
Milestone
Future

Comments

@am11
Copy link
Member

am11 commented Oct 8, 2022

There was a discussion about rewriting ilasm/ildasm in C#. If that is not happening in a near future, it would be nice to combine ilasm/main.cpp and ildasm/windasm.cpp main() functions and the arguments parsing code (1.5K LOC), to make the PAL-like code refactoring easier.

Originally posted by @am11 in #76771 (comment)

cc @AaronRobinsonMSFT, @jkoritzinsky
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Oct 8, 2022
@ghost
Copy link

ghost commented Oct 8, 2022

Tagging subscribers to this area: @JulieLeeMSFT
See info in area-owners.md if you want to be subscribed.

Issue Details

There was a discussion about rewriting ilasm/ildasm in C#. If that is not happening in a near future, it would be nice to combine ilasm/main.cpp and ildasm/windasm.cpp main() functions and the arguments parsing code (1.5K LOC), to make the PAL-like code refactoring easier.

Originally posted by @am11 in #76771 (comment)

cc @AaronRobinsonMSFT, @jkoritzinsky

Author: am11
Assignees: -
Labels:

area-ILTools-coreclr, untriaged
Milestone: -

@JulieLeeMSFT JulieLeeMSFT added needs-further-triage Issue has been initially triaged, but needs deeper consideration or reconsideration and removed untriaged New issue has not been triaged by the area owner labels Oct 10, 2022
@TIHan TIHan added Priority:3 Work that is nice to have and removed needs-further-triage Issue has been initially triaged, but needs deeper consideration or reconsideration labels Apr 24, 2023
@TIHan TIHan added this to the Future milestone May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AaronRobinsonMSFT AaronRobinsonMSFT

Labels
area-ILTools-coreclr Priority:3 Work that is nice to have
Projects
None yet
Milestone
Future
Development

No branches or pull requests
4 participants
@TIHan @am11 @AaronRobinsonMSFT @JulieLeeMSFT