Stream: git-wasmtime

Topic: wasmtime / PR #12932 aarch64: Disable csdb emission by de...


view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 20:36):

alexcrichton opened PR #12932 from alexcrichton:macos-csdb to bytecodealliance:main:

This has a massive performance penalty on macOS, for example, and peer compilers are not emitting this as part of on-by-default mitigations. This commit preserves the option to emit it with an aarch64-specific use_csdb flag, but the default is now false meaning that this is not emitted by default.

Closes #12789

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 20:36):

alexcrichton requested fitzgen for a review on PR #12932.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 20:36):

alexcrichton requested wasmtime-compiler-reviewers for a review on PR #12932.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 21:09):

alexcrichton updated PR #12932.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 21:11):

fitzgen submitted PR review:

Maybe update the spectre docs too?

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 21:24):

alexcrichton requested cfallin for a review on PR #12932.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 21:24):

alexcrichton requested wasmtime-core-reviewers for a review on PR #12932.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 21:24):

alexcrichton requested wasmtime-default-reviewers for a review on PR #12932.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 21:24):

alexcrichton updated PR #12932.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 21:37):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 21:37):

cfallin created PR review comment:

I may be missing it but I don't see a use-site for this term anywhere (and I don't think we should need it since the actual site, which you got, is in the middle of the monolithic JTSequence pseudo-inst) -- can we remove it if so?

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 22:06):

alexcrichton updated PR #12932.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 22:06):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 22:06):

alexcrichton created PR review comment:

Oh oops nah just juggling too much, forgot to add the use of it!

(a lot more tests now got updates)

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 22:06):

alexcrichton has enabled auto merge for PR #12932.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 22:16):

alexcrichton added PR #12932 aarch64: Disable csdb emission by default to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 22:59):

alexcrichton merged PR #12932.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 01 2026 at 22:59):

alexcrichton removed PR #12932 aarch64: Disable csdb emission by default from the merge queue.


Last updated: Apr 12 2026 at 23:10 UTC