bytecodealliance/wasmtime

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

Chiusa

#7588 aperta il 27 nov 2023

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Rust (1531 fork)batch import
good first issue

Metriche repository

Star
 (17.002 stelle)
Metriche merge PR
 (Merge medio 23h 31m) (212 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina il file `cranelift/jit/src/compiled blob.rs`, in particolare le righe 55-57 e il codice circostante, per comprendere come viene utilizzato il puntatore `what` nelle funzioni di rilocazione. Identifica tutte le occorrenze in cui i puntatori vengono convertiti in `usize` o `isize` dopo i calcoli di offset. Implementa una modifica che converta il puntatore base in `usize` in anticipo per evitare blocchi `unsafe` per le chiamate di offset del puntatore.
Tech stack
rustwasm
Dominio
backenddeveloper experience
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rustunsafe Rustpointer arithmetic
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40