-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Allow promotion of structs with single float fields #84627
Allow promotion of structs with single float fields #84627
Conversation
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch, @kunalspathak Issue Detailsnull
|
CC. @dotnet/jit-contrib |
Some good wins on all platforms, some much more than others depending on the ABI. A few small regressions where promotion ends up causing more codegen, but this is the same as with promotion for any other scenario as well. |
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.
LGTM. I think you should run some stress jobs.
/azp run runtime-coreclr outerloop, runtime-coreclr jitstress, runtime-coreclr libraries-jitstress, Fuzzlyn |
Azure Pipelines successfully started running 4 pipeline(s). |
is that the only github issue we have for floating-point-field promotion? I thought there were more 🙂 |
I did too, but GitHub search leaves a lot to be desired and I wasn't able to find them :( |
Logged the following for the two fuzzlyn failures:
For the JitStress failures: |
Outerloop failures are also failing in main. |
This resolves #79928