-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
enable type-based alias analysis for NativeInterpreter
#47574
base: master
Are you sure you want to change the base?
Conversation
4a959af
to
a8f2af5
Compare
@nanosoldier |
f959acd
to
fe5a54a
Compare
a8f2af5
to
1b18773
Compare
Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. |
fe5a54a
to
6ac900c
Compare
1b18773
to
016ee72
Compare
6ac900c
to
374b3f1
Compare
016ee72
to
ce4319a
Compare
@nanosoldier |
Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. |
374b3f1
to
a1fa7e3
Compare
ce4319a
to
4a767de
Compare
4a767de
to
2fb9b70
Compare
@nanosoldier |
Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. |
#41199 implements the base logic of type-based alias analysis with new lattice element `MustAlias`, but doesn't enable it for `NativeInterpreter`. This PR enables it for our native code execution pipeline, checking its performance impact.
2fb9b70
to
fdbb3f9
Compare
@nanosoldier |
Your package evaluation job has completed - possible new issues were detected. A full report can be found here. |
#41199 implements the base logic for new lattice element
MustAlias
, but doesn't enable it forNativeInterpreter
.This PR enables it for our native code execution pipeline, checking its performance impact.