A PR combining the PRs of all Google Code-in students related to loop-invariate optimizer #163
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Jun 30, 2020
jcrodriguez1989
added a commit
that referenced
this pull request
Jul 14, 2021
* Create optimization_ideas.Rmd (#156) Creates optimization_ideas vignette * DCE: examples and Idea section from GCI-2019 DCE: examples and Idea section from GCI-2019 * A PR combining the PRs of all Google Code-in students related to loop-invariate optimizer (#163) thanks! * GCI PR for constant folding optimizer (#164) Co-authored-by: Stephanie <20eristoffs@asij.ac.jp> Co-authored-by: Umang Majumder <54509868+ZeroDawn0D@users.noreply.github.com> Co-authored-by: udaydatar7 <54199456+udaydatar7@users.noreply.github.com> Co-authored-by: Eric Liu <electrobyt1337@gmail.com> Co-authored-by: Pratish <pratishrai724@gmail.com> Co-authored-by: Rahul Saxena <saxenism@gmail.com> * GCI PRs for opt-common-subexpr (#165) thanks! Co-authored-by: Adrian Serapio <adtserapio@gmail.com> Co-authored-by: azamlynny <azamlynny@hotmail.com> Co-authored-by: Umang Majumder <54509868+ZeroDawn0D@users.noreply.github.com> Co-authored-by: Jack-horwell <36800721+Jack-horwell@users.noreply.github.com> Co-authored-by: Pratish <pratishrai724@gmail.com> Co-authored-by: udaydatar7 <54199456+udaydatar7@users.noreply.github.com> Co-authored-by: Juan Cruz Rodriguez <jcrodriguez@unc.edu.ar> Co-authored-by: Rahul Saxena <saxenism@gmail.com> * All GCI PRs for constant propagation optimizer (#166) thanks! Co-authored-by: Pratish <pratishrai724@gmail.com> Co-authored-by: udaydatar7 <54199456+udaydatar7@users.noreply.github.com> Co-authored-by: Umang Majumder <54509868+ZeroDawn0D@users.noreply.github.com> Co-authored-by: Juan Cruz Rodriguez <jcrodriguez@unc.edu.ar> Co-authored-by: Rahul Saxena <saxenism@gmail.com> * All GCI PRs for dead expression optimizer. (#167) thanks! Co-authored-by: Adrian Serapio <adtserapio@gmail.com> Co-authored-by: Umang Majumder <54509868+ZeroDawn0D@users.noreply.github.com> Co-authored-by: Pratish <pratishrai724@gmail.com> Co-authored-by: Eric Liu <electrobyt1337@gmail.com> Co-authored-by: Rahul Saxena <saxenism@gmail.com> * All GCI PRs for Dead Store Optimizer (#168) thanks! Co-authored-by: Umang Majumder <54509868+ZeroDawn0D@users.noreply.github.com> Co-authored-by: Eric Liu <electrobyt1337@gmail.com> Co-authored-by: Pratish <pratishrai724@gmail.com> Co-authored-by: Adrian Serapio <adtserapio@gmail.com> Co-authored-by: Rishi R <56316487+Rishi0812@users.noreply.github.com> Co-authored-by: Rahul Saxena <saxenism@gmail.com> Co-authored-by: Juan Cruz Rodriguez <jcrodriguez@unc.edu.ar> * Conditional Thread Optimizer PR (#162) * Explained Dead Code Elimination * Update opt-dead-code.R * Update opt-dead-code.R * Create opt-dead-code-example.R * Update opt-dead-code.R * Added examples Examples of unoptimized and optimized code to explain dead code elimination * Added #' to example * Non-technical description of dead code elimination * Update to definition in ##Idea GCI user ID Rishi0812 * Added example of Dead Code Elimination * Added Idea Section to Dead Code Elimination Vignette * Updating NEWS.md * Changing \donttest for \dontrun, it seems 4.0.0 will check donttest examples. * Rebuilding site. * Removing renv. * Removing a separate example file for DCE This was introduced when I merged a PR from a student's fork of the `rco`. * Finalizing the Idea section of the DCE * Selected final examples for DCE * Final example selected * Resolved spacing issue * Resolved Spacing Issue - 2 * Resolved Spacing Issue - 3 * Improved the Idea section * Fixed the performance typo * Revert "Fixed the performance typo" This reverts commit c791bef. * Update opt-dead-code.Rmd typo fix * Added a few functions * Added a few functions * Added logic for duplicate expr * Meh * Added merge logic * The entire process of merging implemented. Phhew!! * Ran into the else parse bug * First iteration complete * Final commit for cond_thread * Meh * Resolved duplicate vignette * Solved another issue in vignette build * I'm sleepy now, this is last * Missed a comma :( * I dunno what's happening anymore * Fixed the >= function logic * Last ditch * Ok I'm officially crying now T_T * Now no test would fail * Seriously, this is the last time * Bhak * erfg * Update parse.R * Update opt-constant-folding.R * Update opt-dead-code.R * Dude c'mmon * Update optimize-files.R * Update opt-dead-code.R * plixx * Added more tests to increase coverage * Another test for extra coverage * Implemented suggested changes * Some missing edits * Improved Tests * Fixed boolean condition * Improved indentations * documented * Workaround for R CMD check failed * Version bump & code styling * Examples edit * Fixed the indentation bug * Code Styiling * Provided another example in vignette * Improving vignette * Styling test cases * Fixed unwanted changes due to `;` * Ran all checks, works fine Co-authored-by: Pratish <pratishrai724@gmail.com> Co-authored-by: Ian Chang <ianchangai1339@gmail.com> Co-authored-by: Jack-horwell <36800721+Jack-horwell@users.noreply.github.com> Co-authored-by: Rishi R <56316487+Rishi0812@users.noreply.github.com> Co-authored-by: Adrian Serapio <adtserapio@gmail.com> Co-authored-by: Juan Cruz Rodriguez <jcrodriguez@unc.edu.ar> Co-authored-by: Rahul Saxena <saxenism@gmail.com> * Adding the Memory Allocation Optimizer (#169) * Initial Commit * Second commit - firm logic in place * First iteration of a working prototype * Just another test text * Tests and Vignettes for mem-alloc opt * Final edits in diff files for the new optimizer * Removed unnecessary files * Reverted changes in man/opt_dead * Reverted changes in man/opt_dead * Reverted changes in man/const_prop * Reverted changes in man/common_subexpr Co-authored-by: Rahul Saxena <saxenism@gmail.com> Co-authored-by: Juan C Rodriguez <jcrodriguez@unc.edu.ar> * Possible fix for Issue#107 (#170) * Possible fix for issue#107 * Reverting unrelated changes * Reverting unrelated changes * Reverting unrelated changes Co-authored-by: Rahul Saxena <saxenism@gmail.com> * Added the reporting functions (#171) * Added the reporting functions * Added comments Co-authored-by: Rahul Saxena <saxenism@gmail.com> * Please add this doc in the Article section of the official rco website (#172) * Please add this in the Article section of the official rco website * Fixed some blockquote issues * Trying new changes * Delete Document.dcf Co-authored-by: Rahul Saxena <saxenism@gmail.com> Co-authored-by: Juan C Rodriguez <jcrodriguez@unc.edu.ar> * Checking merge Co-authored-by: Rahul Saxena <rahulsaxena.pro@gmail.com> Co-authored-by: Stephanie <20eristoffs@asij.ac.jp> Co-authored-by: Umang Majumder <54509868+ZeroDawn0D@users.noreply.github.com> Co-authored-by: udaydatar7 <54199456+udaydatar7@users.noreply.github.com> Co-authored-by: Eric Liu <electrobyt1337@gmail.com> Co-authored-by: Pratish <pratishrai724@gmail.com> Co-authored-by: Rahul Saxena <saxenism@gmail.com> Co-authored-by: Adrian Serapio <adtserapio@gmail.com> Co-authored-by: azamlynny <azamlynny@hotmail.com> Co-authored-by: Jack-horwell <36800721+Jack-horwell@users.noreply.github.com> Co-authored-by: Rishi R <56316487+Rishi0812@users.noreply.github.com> Co-authored-by: Ian Chang <ianchangai1339@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.