alexcrichton removed the review request for wasmtime-compiler-reviewers from PR #14907.
alexcrichton opened PR #14907 from alexcrichton:allow-setting-falase to bytecodealliance:main:
This is an API change for
MemFlagsto allow disabling flags in addition to disabling them. This came up during #14683 as a possibly useful primitive to have, and subjectively I've found this sort of builder/setting configuration pretty flexible to use historically too.<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease review the Bytecode Alliance's AI tool usage policy at
https://github.com/bytecodealliance/governance/blob/main/AI_TOOL_POLICY.mdPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
alexcrichton requested wasmtime-compiler-reviewers for a review on PR #14907.
alexcrichton requested cfallin for a review on PR #14907.
github-actions[bot] added the label cranelift on PR #14907.
github-actions[bot] added the label cranelift:area:machinst on PR #14907.
:thumbs_up: cfallin submitted PR review.
cfallin added PR #14907 Add a enabled: bool parameter to MemFlags modifiers to the merge queue.
:check: cfallin merged PR #14907.
cfallin removed PR #14907 Add a enabled: bool parameter to MemFlags modifiers from the merge queue.
Last updated: Oct 31 2026 at 23:03 UTC