-
-
Notifications
You must be signed in to change notification settings - Fork 377
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
Optimize Scalar.As #988
Optimize Scalar.As #988
Conversation
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 to me, although that isn't legal advice i'm not a doctor i think at least
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.
I'll check the disasm and merge. I'm assuming this is good, but blocking merge until we can be sure.
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.
Only checked discord now and saw you've already done everything 😅
b191fcc
to
86b816e
Compare
86b816e
to
ca43f14
Compare
I'm hoping one day this will light up, but failing that I need to add a "mono path" that stops mono from producing invalid assembly (basically just having a file with the old, unoptimized code in it. I think our blazor activity gives us reasonable evidence that it does try to specialize these generics even with the code unoptimized for CoreCLR so we may be good to do so anyway. sucks, but that's mono for you) |
* Optimize Scalar.As * Optimize Scalar.As * Fix mono assembler errors * Fix me spending too much time around C++ * Fix public API * Fix public API (?)
No description provided.