-
-
Notifications
You must be signed in to change notification settings - Fork 802
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
feat[venom]: store expansion pass #4068
feat[venom]: store expansion pass #4068
Conversation
extract IRLiterals which are instruction arguments; this reduces pressure on the stack scheduler
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4068 +/- ##
===========================================
- Coverage 91.36% 46.07% -45.29%
===========================================
Files 108 109 +1
Lines 15662 15691 +29
Branches 3446 3449 +3
===========================================
- Hits 14309 7230 -7079
- Misses 922 7906 +6984
- Partials 431 555 +124 ☔ View full report in Codecov by Sentry. |
also incidentally fixes #4215 |
they do pretty much the same things
we no longer need to keep track of emitted ops
it no longer changes bytecode
use equivalence analysis to reduce swaps
What I did
How I did it
How to verify it
Commit message
Description for the changelog
Cute Animal Picture