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

Sporadic panics on WeakMap tests #2732

Closed
jedel1043 opened this issue Mar 23, 2023 · 2 comments · Fixed by #3052
Closed

Sporadic panics on WeakMap tests #2732

jedel1043 opened this issue Mar 23, 2023 · 2 comments · Fixed by #3052
Labels
bug Something isn't working E-Medium Medium difficulty problem help wanted Extra attention is needed

Comments

@jedel1043
Copy link
Member

So, apparently our CI has some sporadic panics on the following tests:

  • test/built-ins/WeakSet/prototype/delete/delete-entry-initial-iterable.js
  • test/built-ins/WeakSet/prototype/has/returns-false-when-object-value-not-present.js

image

However, I tried to reproduce it in my pc without success, so a platform dependent bug maybe?

Build environment:

  • OS: Arch Linux x86_64
  • Version: 6.2.7-arch1-1
  • Target triple: x86_64-unknown-linux-gnu
  • Rustc version: rustc 1.68.0 (2c8cc3432 2023-03-06)
@jedel1043 jedel1043 added bug Something isn't working help wanted Extra attention is needed E-Medium Medium difficulty problem labels Mar 23, 2023
@langston-barrett
Copy link

Potentially related to #2670.

@jedel1043
Copy link
Member Author

Found a more reliable reproduction of the issue:

let list = []

for (let i = 0; i < 35; i++) {
  list.push(list);
}

let ws = new WeakSet(list);

This snippet always panics for me, which seems to imply that the cause of this is a garbage collection pass within a borrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working E-Medium Medium difficulty problem help wanted Extra attention is needed
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants