-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Revert "Optimize BOX+UNBOX for "T? -> T" and "T -> T?" (#104931)" #105515
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
Revert "Optimize BOX+UNBOX for "T? -> T" and "T -> T?" (#104931)" #105515
Conversation
This reverts commit 9df4f7f.
|
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch |
|
fix for errors in dotnet/efcore#34278 |
|
I needed to add back eeIsSharedInst as that was used by f8bcd05 |
|
Treating as "tell-mode" to unblock preview7. |
|
If it's a direct revert, would you like me to fast-merge it? Edit: Nevermind, I see the extra commit. But let me know anyway if you'd like it merged asap. |
|
Fixed in Main via #105529 |
This reverts commit 9df4f7f. #104931
This change was causing failures in EF Core:
dotnet/efcore#34278 (comment)
https://dev.azure.com/dnceng-public/public/_build/results?buildId=754823&view=results