Skip to content
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

[ValueError@identity_elimination.py:278] ValueError: There are at least two definitions in the identity group var_58#0 = var_58#0 + (&(var_58#0)), var_58#0. #413

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 5, 2024

closes #390

@github-actions github-actions bot added bug Something isn't working bugfinder labels Jun 5, 2024
@NeoQuix NeoQuix force-pushed the issue-390-_ValueError_identity_elimination_py_278_ValueError_There_are_at_least_two_definitions_in_the_identity_group_var_58#0_=_var_58#0_+_&_var_58#0_var_58#0 branch from 2b182a0 to 3ea3a9b Compare June 11, 2024 10:01
@NeoQuix
Copy link
Collaborator

NeoQuix commented Jun 11, 2024

Update: Problem was that the definition is inside a cast (for the same size, so technically unnecessary) wherefore second the check fails (e.g. sub.operand != definition.destination)

@NeoQuix NeoQuix marked this pull request as ready for review June 11, 2024 10:43
@ebehner ebehner requested a review from mari-mari June 24, 2024 08:24
…y_278_ValueError_There_are_at_least_two_definitions_in_the_identity_group_var_58#0_=_var_58#0_+_&_var_58#0_var_58#0
@mari-mari mari-mari merged commit 9647abc into main Jun 25, 2024
2 checks passed
@mari-mari mari-mari deleted the issue-390-_ValueError_identity_elimination_py_278_ValueError_There_are_at_least_two_definitions_in_the_identity_group_var_58#0_=_var_58#0_+_&_var_58#0_var_58#0 branch June 25, 2024 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working bugfinder
Projects
None yet
2 participants