Skip to content

[GC][Threads] Atomic GC operations require --enable-threads#7185

Merged
kripken merged 2 commits intoWebAssembly:mainfrom
kripken:atomic.gc.feature
Jan 6, 2025
Merged

[GC][Threads] Atomic GC operations require --enable-threads#7185
kripken merged 2 commits intoWebAssembly:mainfrom
kripken:atomic.gc.feature

Conversation

@kripken
Copy link
Copy Markdown
Member

@kripken kripken commented Jan 2, 2025

Without this, it is invalid to lower them to simpler atomic operations like
atomic.fence (as some passes do) or linear memory atomics (as a future
lowering pass might do).

Fixes #7184

Copy link
Copy Markdown
Member

@tlively tlively left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@kripken kripken merged commit fcea593 into WebAssembly:main Jan 6, 2025
@kripken kripken deleted the atomic.gc.feature branch January 6, 2025 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GC+atomics and features

2 participants