-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add an argument to SuperPMI #62188
Comments
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
Tagging subscribers to this area: @JulieLeeMSFT Issue DetailsI want to suggest adding an optional -c argument for superpmi.py asmdiffs What will it give?This will allow you to run certain tests, and not all at once, this is especially useful when a problem occurs in a certain test and it is easier to run it alone than to run it all at once. This would especially affect when collecting JitDump. How will it work?Adding a new argument -c to superpmi.py and we throw it into superpmi.exe when we run asmdiff, if -c is specified, then we remove the -p key (since this does not work if -c is specified) I can implement it
|
CC @dotnet/jit-contrib. |
I've wanted to do this for a while; would be happy to see it. Without it, I've always just manually copy/pasted the superpmi.exe command line from the superpmi.log file generated by superpmi.py, and altered it as appropriate. Note that you should add |
I want to suggest adding an optional -c argument for superpmi.py asmdiffs
What will it give?
This will allow you to run certain tests, and not all at once, this is especially useful when a problem occurs in a certain test and it is easier to run it alone than to run it all at once. This would especially affect when collecting JitDump.
How will it work?
Adding a new argument -c to superpmi.py and we throw it into superpmi.exe when we run asmdiff, if -c is specified, then we remove the -p key (since this does not work if -c is specified)
I can implement it
The text was updated successfully, but these errors were encountered: