dotnet/runtime

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

Open

#10 755 ouverte le 23 juil. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C# (5 445 forks)batch import
arch-arm32area-CodeGen-coreclrenhancementhelp wanted

Métriques du dépôt

Stars
 (17 886 stars)
Métriques de merge PR
 (Merge moyen 12j 11h) (661 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de `instGen Set Reg To Imm` dans `src/jit/codegenarm.cpp`. Comprenez la différence entre les immédiats relogeables et non relogeables. Divisez la fonction en deux : une qui accepte un `BYTE* addr` pour les immédiats relogeables, et une autre qui accepte un `target ssize t imm` pour les immédiats non relogeables. Mettez à jour tous les sites d'appel en conséquence.
Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++JIT compiler basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80