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.

Sign up for GitHub

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

Jump to bottom

Sets.incl Has Bug With VM (?) #11083

Closed
loloicci opened this issue Apr 22, 2019 · 2 comments
Closed

Sets.incl Has Bug With VM (?) #11083

loloicci opened this issue Apr 22, 2019 · 2 comments
Labels
Compiler Crash VM see also `const` label

Comments

@loloicci
Copy link
Contributor

Sets.incl sometimes issue Error: attempt to access a nil address or SIGSEGV: Illegal storage access. (Attempt to read from nil?) if it works in VM.

Example

See
loloicci/nimly#24 (comment)
(Sorry for I cannot make some easy examples)

Additional Information

version
Nim: fabc2a7
@ghost
Copy link

ghost commented Apr 22, 2019

It happens when you incl a set inside a seq. With a quick look at your code it seems to affect sets in tables too? See #10981

@mratsim mratsim added Compiler Crash VM see also `const` label labels Apr 22, 2019
@krux02
Copy link
Contributor

krux02 commented Apr 23, 2019

I am closing this right now as it I see it as a duplicate of #10981. Please reply there if you have any additional information that isn't mentioned there.

@krux02 krux02 closed this as completed Apr 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Compiler Crash VM see also `const` label
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@krux02 @mratsim @loloicci