-
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
[SuperPMI] Added a new optional -compile argument #62217
Conversation
…ntexts need to be compiled
Tagging subscribers to this area: @JulieLeeMSFT Issue DetailsBased on #62188 I added an optional Using: With a range: Without -compile: Do I need to make a check for compliance with the format or can I omit this point in the script?
|
@jakobbotsch @BruceForstall please review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good; one suggestion.
Co-authored-by: Bruce Forstall <brucefo@microsoft.com>
Closes #62188
I added an optional
-compile
argument which is just directly thrown into superpmi.exeUsing:
With an index:
py .\superpmi.py asmdiffs-jit_name clrjit_universal_arm64_x64.dll --altjit -target_os Linux -target_arch arm64 -arch x64 -filter coreclr_tests -compile 1
With a range:
py .\superpmi.py replay -jit_name clrjit_universal_arm64_x64.dll --altjit -target_os Linux -target_arch arm64 -arch x64 -filter coreclr_tests -compile 1-200
Without -compile:
py .\superpmi.py asmdiffs-jit_name clrjit_universal_arm64_x64.dll --altjit -target_os Linux -target_arch arm64 -arch x64 -filter coreclr_tests
Do I need to make a check for compliance with the format or can I omit this point in the script?