-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Intrinsic misuse causes segfault #17538
Comments
Oscar (https://github.com/JuliaLang/julia/compare/ob/interp_call) and I (https://github.com/JuliaLang/julia/compare/jn/jlinterpreter) have fixed this, but we have just been bad about getting it onto master. |
|
Well, this issue wasn't about |
Sorry for premature closing and thanks for checking. |
No problem, thanks for working your way through stale issues! |
Since 7a6a9e0#diff-1c5f176b087e9647926b0146fd80117eL823 from #13485 the number of arguments to an intrinsic isn't checked anymore, commonly resulting in segfaults when passing too few of them:
While on 0.4:
I'm sure there's a plan forward here, and while intrinsic aren't meant to be used without care it doesn't seem like a good plan to just have them fault where there used to be some form of error reporting.
cc @vtjnash
The text was updated successfully, but these errors were encountered: