-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[VM] Immutable proc argument is incorrectly modified #19464
Labels
VM
see also `const` label
Comments
capocasa
pushed a commit
to capocasa/Nim
that referenced
this issue
Mar 31, 2023
…im-lang#16780 nim-lang#16613 nim-lang#14553 nim-lang#19909 nim-lang#18641 (nim-lang#19902) [backport] * revert nim-lang#12217 since the root problem seems to have been fixed; fix nim-lang#15974;fix nim-lang#12551; fix nim-lang#19464 * fix nim-lang#16020; fix nim-lang#16780 * fix tests and nim-lang#16613 * fix nim-lang#14553 * fix nim-lang#19909; skip skipRegisterAddr * fix nim-lang#18641
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Example
Current Output
Expected Output
Additional Information
The problem seems to be that
myEcho
directly modifiesa
, instead of just thetmp
copy.This doesn't occur if
assign
(This example may seem contrived, but that's because I tried to reduce it as much as possible. I originally encountered this bug while adding debug
echo
s in nim-lang/bigints.)The text was updated successfully, but these errors were encountered: