-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Fix optSwitchConvert #92249
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
Fix optSwitchConvert #92249
Conversation
|
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch Issue DetailsFixes a last-minute typo. Closes #92246
|
|
/azp run runtime-coreclr outerloop, runtime-coreclr libraries-jitstress |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
@BruceForstall or @jakobbotsch can you please take a look? should unblock outerloop or anyone else @dotnet/jit-contrib small diff: https://dev.azure.com/dnceng-public/public/_build/results?buildId=410576&view=ms.vss-build-web.run-extensions-tab (where we know emit a small/different 64bit constant for bt test) |
|
Why are there 0-sized diffs with no textual JitDisasm diff? Can you check some of those cases? |
The optimization converts series of the bug is basically that the bitmap vector (that 64bit constant bt works with) could become different (and be less than |
You mean the constant would be different but would still display as |
|
Is this a case where we JitDump |
|
Since this fixes widespread pipeline failures, I'm going to merge it. |
Fixes a last-minute typo.
Closes #92246
Closes #92202