dotnet/runtime

Simplify JIT-EE interface modification

Open

#76.184 aberto em 26 de set. de 2022

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)C# (5.445 forks)batch import
area-CodeGen-coreclrhelp 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

Despite the fact it was partially automated in the past it's still quite exhausting to add new APIs, I propose to address at least these issues:

  • Autogenerate corinfo.h (e.g. introduce corinfo_generated.h) part from ThunkInput.txt - we can move comments to ThunkInput.txt
  • Autogenerate superpmi-shim-simple\icorjitinfo.cpp
  • Autogenerate superpmi-shim-collector\icorjitinfo.cpp
  • Append _generated.h to all auto-generated files (currently there are still like 5 or 6 auto-generated files without them - it helps to ignore them when adding a new API) - https://github.com/dotnet/runtime/pull/77083
  • gen.bat emits two new-lines in the end of CorInfoBase.cs breaking compilation - https://github.com/dotnet/runtime/pull/77083
  • Ensure gen.bat is no-op on unchanged codebase on CI

category:cq theme:jit-ee-interface skill-level:beginner cost:small impact:small

Guia do colaborador

Direção de pesquisa
Analise o script gen.bat e ThunkInput.txt para entender o processo manual atual de adicionar novas APIs e, em seguida, projete a geração automatizada para os arquivos listados.
Pilha de tecnologia
csharp
Domain
backendbuild system
Tipo Issue
Funcionalidade
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
C#Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75