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

[ETC] Enhance instance extraction to extract dependent instances. #4910

Merged
merged 2 commits into from
Apr 4, 2023

Conversation

mikeurbach
Copy link
Contributor

We previously extracted instances that only feed the test code. This change enhances that logic to also extract instances that may feed other instances, which transitively only feed the test code. This also works for instances that potentially feed each other, cyclicly, and otherwise only feed the test code.

We previously extracted instances that only feed the test code. This
change enhances that logic to also extract instances that may feed
other instances, which transitively only feed the test code. This also
works for instances that potentially feed each other, cyclicly, and
otherwise only feed the test code.
Copy link
Member

@seldridge seldridge left a comment

Choose a reason for hiding this comment

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

LGTM as far as I understand this part of the codebase.

}

hw.output %0 : i1
}
Copy link
Member

Choose a reason for hiding this comment

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

Nice, succinct test!

@mikeurbach mikeurbach merged commit 3dcf278 into main Apr 4, 2023
@mikeurbach mikeurbach deleted the mikeurbach/etc-extract-dependent-instances branch April 4, 2023 02:09
uenoku pushed a commit that referenced this pull request Apr 14, 2023
)

We previously extracted instances that only feed the test code. This
change enhances that logic to also extract instances that may feed
other instances, which transitively only feed the test code. This also
works for instances that potentially feed each other, cyclicly, and
otherwise only feed the test code.
uenoku pushed a commit that referenced this pull request Apr 14, 2023
)

We previously extracted instances that only feed the test code. This
change enhances that logic to also extract instances that may feed
other instances, which transitively only feed the test code. This also
works for instances that potentially feed each other, cyclicly, and
otherwise only feed the test code.
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