-
Notifications
You must be signed in to change notification settings - Fork 50
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
Track more precisely tuple to tuple assignments in sim #667
Conversation
@bgregoir Unfortunately it seems that now that we introduce intermediate variables,
results in
(where Do you see what's wrong with the code? Or do you have suggestions on how to debug this ? |
I think the order of the list should be reversed (remember rev) : |
4920255
to
41b360d
Compare
Thanks that was the issue indeed. Now all gimli CT proof work. |
Now you can criticize me (constructively) for using the variable name hack. |
I would love to do that. Unfortunately I don't know how we can propose a nicer solution without an important modification of the code. We need to have the memories of the two current functions to be able to create fresh names properly. |
I'm fine with leaving this as-is, if it is sound. |
I think we should use this function : |
41b360d
to
194bd78
Compare
@bgregoir This is now ready. |
194bd78
to
3d95c3c
Compare
Fixes #666