Wasmtime GitHub notifications bot (Apr 24 2024 at 21:02): Wasmtime GitHub notifications bot (Apr 24 2024 at 21:02):jameysharp opened PR #8467 from jameysharp:x64-into-reg to bytecodealliance:main:
Previously, the operand collector required its arguments to be of type Reg. But the x86-64 backend uses newtype wrappers around Reg to ensure that different register classes don't get mixed up, so it had to convert types everywhere to pass registers to the operand collector.
This commit allows the operand collector to accept any type which implements
Into<Reg>instead. The newtype wrappers already implemented that, so we can mostly just delete theto_regandto_writable_regcalls.However, those calls were also automatically dereferencing borrows as needed, so to minimize the need for explicit dereferences I've also introduced implementations for converting from &Gpr/&Xmm to Reg. That doesn't help with the Writable-wrapped registers but it's good enough. I have a PR in progress which will clean it up the rest of the way.
Wasmtime GitHub notifications bot (Apr 24 2024 at 21:02):jameysharp requested wasmtime-compiler-reviewers for a review on PR #8467.
Wasmtime GitHub notifications bot (Apr 24 2024 at 21:02):jameysharp requested cfallin for a review on PR #8467.
Wasmtime GitHub notifications bot (Apr 24 2024 at 21:32):cfallin submitted PR review:
So much cleaner; nice!
Wasmtime GitHub notifications bot (Apr 24 2024 at 22:09):jameysharp merged PR #8467.
Last updated: Oct 23 2024 at 20:03 UTC