-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
codegen: optimize setfield/arrayset with inline isa test #41587
Conversation
@nanosoldier |
Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @christopher-dG |
@nanosoldier |
Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @christopher-dG |
@nanosoldier |
This is not a particularly meaningful combination (LLVM dislikes it).
2bd9348
to
471e893
Compare
Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @christopher-dG |
I didn't realize we weren't doing this, since it is quite trivial, and I was assuming it was done for some of the codegen tests for the new atomics. Needed to thus first disable a non-sensical atomic RMW operation and fix a return type.