bytecodealliance/wasmtime

Eagerly cast pointers in aarch64 relocation code to `usize` to avoid `unsafe` blocks in cranelift-jit

Fermée

#7 588 ouverte le 27 nov. 2023

 (2 commentaires) (0 réaction) (0 personne assignée)Rust (1 531 forks)batch import
good first issue

Métriques du dépôt

Stars
 (17 002 étoiles)
Métriques de merge PR
 (Merge moyen 23h 31m) (212 PRs mergées en 30 j)

Description

When performing relocations in cranelift-jit, there are many introductions of a variable what that's defined as an unsafe pointer offset. These variables are almost always cast to usize or isize before they're used, so perhaps we should instead turn the base pointer into its target numeric type eagerly, to avoid needing the unsafe blocks for calling the offset method on the pointers.

https://github.com/bytecodealliance/wasmtime/blob/cd97c9f14713d81de5453fb9861319e78a761432/cranelift/jit/src/compiled_blob.rs#L55-L57

Guide contributeur