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.

Sign up for GitHub

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

Jump to bottom

Don't copy Match Case body for OR patterns in switch-optimized Match Expressions #431

Open
Clashsoft opened this issue Oct 18, 2018 · 0 comments
Open

Don't copy Match Case body for OR patterns in switch-optimized Match Expressions #431

Clashsoft opened this issue Oct 18, 2018 · 0 comments
Assignees
@Clashsoft
Labels
compiler:Internal status:Review type:Enhancement Enhancements to exisiting functionality and small features
Projects
Dyvil

Comments

@Clashsoft
Copy link
Member

Match Expressions that can be switch-optimized currently copy the body of Match Cases with OR Patterns once for every sub-pattern. This can and should be avoided.
@Clashsoft Clashsoft added this to the v0.42.0 milestone Oct 18, 2018
@Clashsoft Clashsoft self-assigned this Oct 18, 2018
@Clashsoft Clashsoft added compiler:Internal type:Enhancement Enhancements to exisiting functionality and small features status:In Progress Implementation in progress status:Scheduled Scheduled for the next release and removed status:In Progress Implementation in progress labels Oct 18, 2018
@Clashsoft Clashsoft removed this from the v0.42.0 milestone Nov 5, 2018
@Clashsoft Clashsoft added status:Review and removed status:Scheduled Scheduled for the next release labels Nov 12, 2018
@Clashsoft Clashsoft added this to Review in Dyvil Apr 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Clashsoft Clashsoft

Labels
compiler:Internal status:Review type:Enhancement Enhancements to exisiting functionality and small features
Projects
Dyvil
  
Review
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Clashsoft