jameysharp requested elliottt for a review on PR #8605.
jameysharp requested wasmtime-compiler-reviewers for a review on PR #8605.
jameysharp opened PR #8605 from jameysharp:no-fixed-nonallocatable
to bytecodealliance:main
:
Consumption of non-allocatable operands was added in #5253 and #5132, and removed in #8524 and following PRs. Now they are not only ignored by regalloc2, but the placeholders that it leaves in the allocation results are also ignored by Cranelift. So let's stop adding them to the operands list entirely.
elliottt submitted PR review:
This was surprising to me! However, if we don't ever generate RA2 operands for VRegs that have already been given allocations and we use those existing allocations during instruction emission, we just don't need the fixed-nonallocatable constraints!
Very cool change :)
jameysharp merged PR #8605.
Last updated: Dec 23 2024 at 12:05 UTC