bytecodealliance/wasmtime

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

Open

#7,588 opened on 2023年11月27日

GitHub で見る
 (2 comments) (0 reactions) (0 assignees)Rust (17,002 stars) (1,531 forks)batch import
good first issue

説明

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

コントリビューターガイド

技術スタック
rustwasm
領域
backenddeveloper experience
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Rustunsafe Rustpointer arithmetic
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40
調査方針
Examine the file `cranelift/jit/src/compiled blob.rs`, specifically lines 55-57 and surrounding code, to understand how the `what` pointer is used in relocation functions. Identify all occurrences where pointers are cast to `usize` or `isize` after offset calculations. Implement a change that casts the base pointer to `usize` early to avoid `unsafe` blocks for pointer offset calls.