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

Blocks of multiple arities are not handled correctly at same send site #21

Closed
smarr opened this issue Jul 27, 2016 · 4 comments
Closed

Blocks of multiple arities are not handled correctly at same send site #21

smarr opened this issue Jul 27, 2016 · 4 comments
Labels
bug Fixes an issue, incorrect implementation language-design Not everything is in the spec, sometimes, we need to decide what's best.
Projects
Completeness
Milestone
v0.6.0 - Black Di...

Comments

@smarr
Copy link
Owner

smarr commented Jul 27, 2016

See charig/TruffleMATE@da10f5f#commitcomment-18420330
@smarr smarr changed the title There might be a bug for send sites with blocks of multiple arities Blocks of multiple arities are not handled correctly at same send site Jul 27, 2016
@smarr
Copy link
Owner Author

smarr commented Jul 27, 2016

Test charig/SOM@3a7e9af

@smarr
Copy link
Owner Author

smarr commented Sep 24, 2016

A proper fix should also handle if not enough arguments are given to block.
At the moment, it triggers a assertion, I think.

@smarr smarr added the bug Fixes an issue, incorrect implementation label Sep 24, 2016
@smarr smarr added the language-design Not everything is in the spec, sometimes, we need to decide what's best. label Jan 18, 2017
@smarr
Copy link
Owner Author

smarr commented Jan 18, 2017

This should be resolved together with #94.

@smarr smarr mentioned this issue Jan 18, 2017
Closed
@smarr smarr added this to the later milestone Aug 7, 2017
smarr added a commit that referenced this issue Feb 17, 2018
@smarr
Test blocks with different arity in same location (closes #21)
59add04 
Signed-off-by: Stefan Marr <git@stefan-marr.de>
@smarr
Copy link
Owner Author

smarr commented Feb 17, 2018

Solved with #215 and verified with 59add04.

@smarr smarr closed this as completed Feb 17, 2018
Completeness automation moved this from Open Issues to Completed Feb 17, 2018
@smarr smarr modified the milestones: later, v0.6.0 - Black Diamonds Feb 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Fixes an issue, incorrect implementation language-design Not everything is in the spec, sometimes, we need to decide what's best.
Projects
No open projects
Completeness
  
Completed
Milestone
v0.6.0 - Black Diamonds
Development

No branches or pull requests
1 participant
@smarr