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

Branch restructuring fail to pick up branches inside loop #47

Closed
sklam opened this issue Apr 18, 2023 · 1 comment
Closed

Branch restructuring fail to pick up branches inside loop #47

sklam opened this issue Apr 18, 2023 · 1 comment

Comments

@sklam
Copy link
Member

sklam commented Apr 18, 2023

From DEBUGGRAPH=1 pytest numba_rvsdg/tests/test_mock_asm.py::test_mock_scfg_double_exchange_loop in #42

CFG:

Screenshot 2023-04-18 at 5 15 51 PM

loop and branch restructured
Screenshot 2023-04-18 at 5 16 01 PM

Missing branch regions inside loop region.

@sklam
Copy link
Member Author

sklam commented Apr 25, 2023

Now i realize that the restructuring functions are not by default recursive and fixed this in #42. This is no longer an issue.

@sklam sklam closed this as completed Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant