Skip to content

Fix loop parent computation in DataFlow.Graph#2522

Merged
aheejin merged 1 commit intoWebAssembly:masterfrom
aheejin:souperify_fix
Dec 11, 2019
Merged

Fix loop parent computation in DataFlow.Graph#2522
aheejin merged 1 commit intoWebAssembly:masterfrom
aheejin:souperify_fix

Conversation

@aheejin
Copy link
Copy Markdown
Member

@aheejin aheejin commented Dec 11, 2019

This fixes the parent-child relationship computation in DataFlow.Graph
when there is a loop. This wasn't discovered until now because this is
used in Souperify and Souperify only runs after Flatten pass, which
produces redundant blocks between inside and outside of a loop.

This fixes the parent-child relationship computation in `DataFlow.Graph`
when there is a loop. This wasn't discovered until now because this is
used in Souperify and Souperify only runs after Flatten pass, which
produces redundant blocks between inside and outside of a loop.
@aheejin aheejin requested a review from kripken December 11, 2019 08:33
Copy link
Copy Markdown
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@aheejin aheejin merged commit acd786d into WebAssembly:master Dec 11, 2019
@aheejin aheejin deleted the souperify_fix branch December 11, 2019 22:49
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

Successfully merging this pull request may close these issues.

2 participants