Skip to content

Implement available expression analysis#503

Merged
seanyoung merged 1 commit intohyperledger-solang:mainfrom
LucasSte:available-expression
Oct 6, 2021
Merged

Implement available expression analysis#503
seanyoung merged 1 commit intohyperledger-solang:mainfrom
LucasSte:available-expression

Conversation

@LucasSte
Copy link
Contributor

@LucasSte LucasSte commented Sep 30, 2021

This PR implements an available expression analysis for the control flow graph of Solang. This analysis is going to be necessary for the common subexpression elimination pass to be implemented.

This PR is part of the Linux mentorship. For more information, please refer to the project's wiki.

@LucasSte LucasSte force-pushed the available-expression branch 3 times, most recently from ec73fd3 to 35b45e6 Compare October 2, 2021 17:45
@LucasSte LucasSte marked this pull request as ready for review October 2, 2021 17:53
@LucasSte LucasSte requested a review from seanyoung as a code owner October 2, 2021 17:53
Copy link
Contributor

@seanyoung seanyoung left a comment

Choose a reason for hiding this comment

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

This could do with some documentation. I think this is original work, after all. Maybe something in docs/optimizer.rst and/or a big comment at the top of the file.

@LucasSte LucasSte force-pushed the available-expression branch 2 times, most recently from ab091cf to b7653db Compare October 6, 2021 12:42
Signed-off-by: Lucas Steuernagel <lucas.tnagel@gmail.com>
@LucasSte LucasSte force-pushed the available-expression branch from b7653db to c791c01 Compare October 6, 2021 12:42
@LucasSte LucasSte requested a review from seanyoung October 6, 2021 12:44
@seanyoung seanyoung merged commit dcf38e2 into hyperledger-solang:main Oct 6, 2021
@LucasSte LucasSte deleted the available-expression branch October 8, 2021 10:31
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