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

Avoid infinite method error on bootstrap error #18438

Merged
merged 1 commit into from
Sep 12, 2016
Merged

Conversation

yuyichao
Copy link
Contributor

Also make the control flow in exec_program easier to understand.

@yuyichao yuyichao merged commit 9a40607 into master Sep 12, 2016
@yuyichao yuyichao deleted the yyc/boot-metherr branch September 12, 2016 01:24
@tkelman
Copy link
Contributor

tkelman commented Sep 16, 2016

is whatever this was fixing also an issue on release-0.5?

@yuyichao
Copy link
Contributor Author

It does. But this is a developer only feature.

@tkelman
Copy link
Contributor

tkelman commented Sep 16, 2016

or if you do something wrong in userimg, right?

@tkelman tkelman added this to the 0.5.x milestone Sep 16, 2016
@yuyichao
Copy link
Contributor Author

No

@tkelman
Copy link
Contributor

tkelman commented Sep 16, 2016

what would cause it then? will wait until 0.5.1

@yuyichao
Copy link
Contributor Author

Raising an uncaught error before it can be shown or essentially anytime when something is broken, which is why I said this is a developer only feature. (In #18444 I think it was when I messed up some parameter packing for vararg functions in type inference).

Might also fix hanging when pressing Ctrl-C too early in sysimg compilation.

tkelman pushed a commit that referenced this pull request Feb 22, 2017
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

Successfully merging this pull request may close these issues.

None yet

2 participants