Add separate flags for each scode optimization #901

Merged
ghallak merged 5 commits from ghallak/more-optimization-flags into master 2022-10-07 18:09:54 +09:00
ghallak commented 2022-08-31 18:34:49 +09:00 (Migrated from gitlab.com)
No description provided.
zxq9 commented 2022-09-16 17:27:24 +09:00 (Migrated from gitlab.com)

Created by: radrow

Either make use of the sections by providing some descriptions or make it, or just use a list

- optimize_inliner
- optimize_inline_local_functions
- optimize_bind_subexpressions
- optimize_let_floating
- optimize_simplifier
- optimize_drop_unused_lets
- optimize_push_consume
- optimize_one_shot_var
- optimize_write_to_dead_var
- optimize_inline_switch_target
- optimize_swap_push
- optimize_swap_pop
- optimize_swap_write
- optimize_constant_propagation
- optimize_prune_impossible_branches
- optimize_single_successful_branch
- optimize_inline_store
- optimize_float_switch_bod
*Created by: radrow* Either make use of the sections by providing some descriptions or make it, or just use a list ```suggestion:-0+0 - optimize_inliner - optimize_inline_local_functions - optimize_bind_subexpressions - optimize_let_floating - optimize_simplifier - optimize_drop_unused_lets - optimize_push_consume - optimize_one_shot_var - optimize_write_to_dead_var - optimize_inline_switch_target - optimize_swap_push - optimize_swap_pop - optimize_swap_write - optimize_constant_propagation - optimize_prune_impossible_branches - optimize_single_successful_branch - optimize_inline_store - optimize_float_switch_bod ```
zxq9 commented 2022-09-16 17:28:44 +09:00 (Migrated from gitlab.com)

Created by: radrow

list_to_existing_atom is strongly preferred -- did you check if it is not an option?

*Created by: radrow* `list_to_existing_atom` is strongly preferred -- did you check if it is not an option?
ghallak commented 2022-10-04 23:03:32 +09:00 (Migrated from gitlab.com)

I don't think that list_to_existing_atom/1 would work here. I am creating the atoms for the first time in this macro, they do not exist anywhere else.

I don't think that `list_to_existing_atom/1` would work here. I am creating the atoms for the first time in this macro, they do not exist anywhere else.
zxq9 commented 2022-10-05 00:47:40 +09:00 (Migrated from gitlab.com)

Created by: radrow

Review: Approved

*Created by: radrow* **Review:** Approved
zxq9 commented 2022-10-07 18:09:10 +09:00 (Migrated from gitlab.com)

Created by: hanssv

Review: Approved

*Created by: hanssv* **Review:** Approved
zxq9 commented 2022-10-07 18:09:54 +09:00 (Migrated from gitlab.com)

Merged by: ghallak at 2022-10-07 09:09:54 UTC

*Merged by: ghallak at 2022-10-07 09:09:54 UTC*
Sign in to join this conversation.
No description provided.