dotnet/runtime

Remove JIT-workaround for eliminating bound check

Open

#67 044 ouverte le 23 mars 2022

Voir sur GitHub
 (12 commentaires) (3 réactions) (1 assigné)C# (5 445 forks)batch import
area-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

https://github.com/dotnet/runtime/pull/62864 taught the JIT to recognize more pattern to eliminate bound checks for array and span indexing. Thus the workarounds like the uint-casts aren't needed anymore, and could be undone.

Also cf. https://github.com/dotnet/runtime/pull/64899#discussion_r833378404

category:cq theme:bounds-checks skill-level:beginner cost:small impact:small

Guide contributeur

Direction de recherche
Ce problème concerne la suppression des contournements (comme les casts en uint) qui ont été introduits pour aider le JIT à éliminer les vérifications de limites. Examinez les PR de référence #62864 et #64899, ainsi que la discussion dans les commentaires. L'objectif est d'identifier les endroits dans la base de code où ces contournements existent et de les restaurer sous des formes plus simples. Concentrez vous sur le code JIT et la logique d'indexation des tableaux/spans.
Stack technique
csharp
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
GitC#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85