dotnet/runtime

Remove JIT-workaround for eliminating bound check

Open

#67.044 aberto em 23 de mar. de 2022

Ver no GitHub
 (12 comments) (3 reactions) (1 assignee)C# (5.445 forks)batch import
area-CodeGen-coreclrenhancementhelp wanted

Métricas do repositório

Stars
 (17.886 stars)
Métricas de merge de PR
 (Mesclagem média 12d 11h) (661 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Esta questão envolve a remoção de soluções alternativas (como conversões para uint) que foram introduzidas para ajudar o JIT a eliminar verificações de limites. Examine os PRs de referência #62864 e #64899 e a discussão nos comentários. O objetivo é identificar locais na base de código onde essas soluções alternativas existem e revertê las para formas mais simples. Concentre se no código do JIT e na lógica de indexação de array/span.
Pilha de tecnologia
csharp
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC#
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85