dotnet/runtime

[ARM32] Split CodeGen::instGen_Set_Reg_To_Imm into relocatable immediate/non-relocatable immediate cases

オープン

#10,755 opened on 2018/07/23

GitHub で見るコントリビューターガイド
 (0 件のコメント) (0 件のリアクション) (0 人の担当者)C# (5,445 件のフォーク)batch import
arch-arm32area-CodeGen-coreclrenhancementhelp wanted

Repository metrics

Stars
 (17,886 個のスター)
PR merge metrics
 (平均マージ 12d 11h) (30d で 661 merged PRs)

説明

As suggested in https://github.com/dotnet/coreclr/pull/19013#discussion_r204171338 the next step complement to dotnet/coreclr#19013 should be splitting instGen_Set_Reg_To_Imm in src/jit/codegenarm.cpp into two separate functions:

  1. the one that accepts only relocatable immediate (as a BYTE* addr);
  2. another one that accepts non-relocatable immediate (as a target_ssize_t imm).

@BruceForstall Please feel free to add if I missed anything

category:implementation theme:emitter skill-level:beginner cost:small impact:small

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

調査方針
`src/jit/codegenarm.cpp` の `instGen Set Reg To Imm` の現在の実装を調べてください。再配置可能な即値とそうでない即値の違いを理解します。関数を2つに分割します。1つは再配置可能な即値用に `BYTE* addr` を受け取り、もう1つは再配置不可能な即値用に `target ssize t imm` を受け取ります。すべての呼び出し元を適切に更新してください。
技術スタック
cpp
領域
backend
Issue 種別
リファクタリング
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C++JIT compiler basics
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
80