-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
SPMI CI runs need to surface context misses #70350
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, @jakobbotsch Issue DetailsIf there are a significant number of misses in SPMI runs then the results we show on the "Extensions" page are very misleading, both for the ASM diffs and TP impact. We need to surface the misses so that we can more easily evaluate whether a normal jit-diff run is needed.
|
cc @dotnet/jit-contrib |
Related, there's a request to surface asserts on the Extensions page as well: #61483 |
* Report the total number of contexts, minopts contexts and fullopts contexted processed * Report number of successful and missing contexts * Report asmdiffs and tpdiffs for minopts/fullopts separately Fixes dotnet#70350 Contributes to dotnet#73506
If there are a significant number of misses in SPMI runs then the results we show on the "Extensions" page are very misleading, both for the ASM diffs and TP impact. We need to surface the misses so that we can more easily evaluate whether a normal jit-diff run is needed.
The text was updated successfully, but these errors were encountered: