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

feat[venom]: strip superfluous instructions from basic blocks #4043

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

harkal
Copy link
Collaborator

@harkal harkal commented May 22, 2024

What I did

This PR removes nops and instructions following revert from blocks

How I did it

How to verify it

Commit message

This commit removes `nop`s and instructions following `revert` from blocks

`SimplifyCFGPass` is a pass that gets to do the miscellaneous simplifications
of basic blocks. This commit strips instructions early to alleviate burden from later
more complex passes. It also removes in-block dead instructions that not only
help later passes with performance, but also is a prerequisite for proper operation
of some of them.

Description for the changelog

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@harkal harkal marked this pull request as ready for review May 22, 2024 09:37
@harkal harkal marked this pull request as draft May 22, 2024 09:50
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.

None yet

2 participants