-
-
Notifications
You must be signed in to change notification settings - Fork 830
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
fix[venom]: fix duplicate allocas #4321
Conversation
|
apart from the misleading comment lgtm. I agree with the commit msg in that it shouldn't be necessary to introduce venom-only path. checked only the legacy version. |
There seems to be small regression on |
i'm guessing it breaks some alias case in mem2var.. but we should get this in for correctness and work separately on mem2var |
Co-authored-by: Harry Kalogirou <harkal@nlogn.eu>
What I did
How I did it
How to verify it
Commit message
Description for the changelog
Cute Animal Picture