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.

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

returning from an if block that has a LEAVE phaser causes a "const_iX NYI" error #4346

Closed
p6rt opened this issue Jun 24, 2015 · 7 comments
Closed

Comments

@p6rt
Copy link

p6rt commented Jun 24, 2015

Migrated from rt.perl.org#125480 (status was 'resolved')

Searchable as RT125480$

@p6rt
Copy link
Author

p6rt commented Jun 24, 2015

From @hoelzro

See the attached script.

This was found via perl6/doc, which uses this technique to clean up temporary files.

The commit responsible is bbc369b.

@p6rt
Copy link
Author

p6rt commented Jun 24, 2015

From @hoelzro

golfed-bad-htmlify.p6

@p6rt
Copy link
Author

p6rt commented Jun 25, 2015

From @hoelzro

On 2015-06-24 13​:16​:32, rob@​hoelz.ro wrote​:

See the attached script.

This was found via perl6/doc, which uses this technique to clean up
temporary files.

The commit responsible is bbc369b.

This works on the JVM, but not on Moar.

@p6rt
Copy link
Author

p6rt commented Jun 25, 2015

From @hoelzro

On 2015-06-25 06​:10​:53, rob@​hoelz.ro wrote​:

On 2015-06-24 13​:16​:32, rob@​hoelz.ro wrote​:

See the attached script.

This was found via perl6/doc, which uses this technique to clean up
temporary files.

The commit responsible is bbc369b.

This works on the JVM, but not on Moar.

jnthn pointed to me that this is probably because the LEAVE block wasn't getting run before, which seems to be the case.

@p6rt
Copy link
Author

p6rt commented Jul 30, 2015

From @jnthn

On Thu Jun 25 14​:42​:46 2015, rob@​hoelz.ro wrote​:

On 2015-06-25 06​:10​:53, rob@​hoelz.ro wrote​:

On 2015-06-24 13​:16​:32, rob@​hoelz.ro wrote​:

See the attached script.

This was found via perl6/doc, which uses this technique to clean up
temporary files.

The commit responsible is bbc369b.

This works on the JVM, but not on Moar.

jnthn pointed to me that this is probably because the LEAVE block
wasn't getting run before, which seems to be the case.

Fixed the MoarVM bug responsible, and added test coverage in S04-phasers/enter-leave.t.

@p6rt
Copy link
Author

p6rt commented Jul 30, 2015

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Jul 30, 2015

@jnthn - Status changed from 'open' to 'resolved'

@p6rt p6rt closed this as completed Jul 30, 2015
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

No branches or pull requests

1 participant