-
-
Notifications
You must be signed in to change notification settings - Fork 419
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
Consume on LHS of assignment causes segfault #3463
Labels
help wanted
Extra attention is needed
Comments
Thanks for the detailed report. I think this is the same assertion as is being hit in #3453. |
SeanTAllen
added
help wanted
Extra attention is needed
bug
and removed
duplicate
labels
May 12, 2020
kapilash
added a commit
to kapilash/ponyc
that referenced
this issue
Sep 11, 2020
* `consume (consume variable).field` must result in similar compiation error as * `consume (consume variable)`. * `consume f().field` must result in similar compilation error as `consume f()` * `(consume f).b` must be treated as `consume f.b` Fixes ponylang#3463, ponylang#3567 (and ponylang#3453)
kapilash
added a commit
to kapilash/ponyc
that referenced
this issue
Sep 12, 2020
* `consume (consume variable).field` must result in a similar compillation error as `consume (consume variable)`. * `consume f().field` must result in similar compilation error as `consume f()` * `(consume f).b` must be allowed and `f` should be moved. Fixes ponylang#3463, ponylang#3567 (and ponylang#3453)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I was experimenting with how Pony handles paths on the LHS of an assignment which have side-effects, and I managed to trip over this compiler bug.
System
The distribution is Parabola, a free Arch variant. Unless packages contain non-free software, they are normally sourced directly from Archlinux instead of being repackaged.
Compiler version
Minimal Example
Expected Result
There's nothing inherently wrong with the code as written, I think - so it should compile, even if it is a bit nonsensical and people should be discouraged from writing it in production.
Compiler Output
The text was updated successfully, but these errors were encountered: