dotnet/runtime

Remove JIT-workaround for eliminating bound check

Open

#67.044 aperta il 23 mar 2022

Vedi su GitHub
 (12 commenti) (3 reazioni) (1 assegnatario)C# (5445 fork)batch import
area-CodeGen-coreclrenhancementhelp wanted

Metriche repository

Star
 (17.886 star)
Metriche merge PR
 (Merge medio 12g 11h) (661 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Questo problema riguarda la rimozione di soluzioni alternative (come i cast a uint) introdotte per aiutare il JIT a eliminare i controlli dei limiti. Esamina i PR di riferimento #62864 e #64899 e la discussione nei commenti. L'obiettivo è identificare i punti della codebase in cui esistono queste soluzioni alternative e ripristinarle in forme più semplici. Concentrati sul codice JIT e sulla logica di indicizzazione di array/span.
Tech stack
csharp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85