Skip to content

Scala 2.13's scala.util.RandomTest.testShuffle() crashes V8 on WebAssembly #4990

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
Open
sjrd opened this issue May 26, 2024 · 2 comments
Open

Scala 2.13's scala.util.RandomTest.testShuffle() crashes V8 on WebAssembly #4990

sjrd opened this issue May 26, 2024 · 2 comments
Labels
internal Not visible to users of Scala.js (only by devs in this repo) wasm Applies to the WebAssembly backend only

Comments

@sjrd
Copy link
Member

sjrd commented May 26, 2024 •
edited
Loading

Filing this ahead of #4988 being merged, but it will only apply once it is.

See the exclusion of scala.util.RandomTest in project/Build.scala when enabling WebAssembly.

We suspect a V8 bug rather than a bug on our side. No matter how bad the code we generate is, V8 shouldn't get into a SIGSEGV.
@sjrd sjrd added internal Not visible to users of Scala.js (only by devs in this repo) wasm Applies to the WebAssembly backend only labels May 26, 2024
@sjrd sjrd mentioned this issue May 26, 2024
Merged
@gzm0
Copy link
Contributor

gzm0 commented May 28, 2024

Hum, the referenced code excludes scala.util.RandomTest, not TryTest.

@sjrd sjrd changed the title Scala 2.13's scala.util.TryTest crashes V8 on WebAssembly Scala 2.13's scala.util.RandomTest.testShuffle() crashes V8 on WebAssembly May 29, 2024
@sjrd
Copy link
Member Author

sjrd commented May 29, 2024

Oops, yes, it's RandomTest. At some point I also fought against TryTest and apparently it overcame my short-term memory when I created this issue. Fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
internal Not visible to users of Scala.js (only by devs in this repo) wasm Applies to the WebAssembly backend only
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sjrd @gzm0