dotnet/runtime

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

Open

#10.755 geöffnet am 23. Juli 2018

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (5.445 Forks)batch import
arch-arm32area-CodeGen-coreclrenhancementhelp wanted

Repository-Metriken

Stars
 (17.886 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 12T 11h) (661 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die aktuelle Implementierung von `instGen Set Reg To Imm` in `src/jit/codegenarm.cpp`. Verstehen Sie den Unterschied zwischen verschiebbaren und nicht verschiebbaren Immediate Werten. Teilen Sie die Funktion in zwei auf: eine, die einen `BYTE* addr` für verschiebbare Immediate akzeptiert, und eine andere, die einen `target ssize t imm` für nicht verschiebbare Immediate akzeptiert. Aktualisieren Sie alle Aufrufstellen entsprechend.
Tech Stack
cpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++JIT compiler basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80