-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
arc devel #12781
arc devel #12781
Conversation
@cooldome please test this branch with your codebase. |
Sure, can do it on Monday |
I wanted to try this one today but I have found that thanks to the #12766 fix, the code now compiles proc add(c: Container, x: sink MyValue) didn't get the destructor injection for Will keep on debugging and compare generated code. |
@cooldome If the current devel cannot compile your projects we might as well go ahead and merge this PR, right? |
Yes, looks like is going to take some time to trace it down. Please submit and we can fix it later. |
Note to the person who will merge this: beware that some of the commits have EDIT: I've just been told that those two commits are already in devel. I'm squashing+merging this then. |
more ARC related bugfixes and refactorings