dotnet/runtime

Simplify JIT-EE interface modification

Open

#76.184 aperta il 26 set 2022

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-CodeGen-coreclrhelp wanted

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
Analizza lo script gen.bat e ThunkInput.txt per comprendere l'attuale processo manuale di aggiunta di nuove API, quindi progetta la generazione automatica per i file elencati.
Tech stack
csharp
Dominio
backendbuild system
Tipo issue
Funzionalità
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
C#Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75