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

Update stackMapping to Offset -> Cell #274

Open
sadrabt opened this issue Nov 25, 2024 · 0 comments
Open

Update stackMapping to Offset -> Cell #274

sadrabt opened this issue Nov 25, 2024 · 0 comments
Assignees

Comments

@sadrabt
Copy link
Contributor

sadrabt commented Nov 25, 2024

Currently stackMapping is implemented as Offset -> Node with cell 0 of the node representing the cell for that offset. This is fine in the local phase since if cell 0 of the stack node gets merged into a different offset of another node, the local graph will track it via the union-find data structure. However, once the graph is cloned, the union-find information is lost and hence stack positions which have been updated to be a cell at an offset different to 0 will be lost.

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