search

dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
15.19k stars 4.72k forks source link

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

Open am11 opened 2 years ago

am11 commented 2 years ago

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 https://github.com/dotnet/runtime/pull/76771#discussion_r990561863_

cc @AaronRobinsonMSFT, @jkoritzinsky

ghost commented 2 years ago

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 https://github.com/dotnet/runtime/pull/76771#discussion_r990561863_ cc @AaronRobinsonMSFT, @jkoritzinsky
Author: am11
Assignees: -
Labels: `area-ILTools-coreclr`, `untriaged`
Milestone: -