dotnet/runtime

[RyuJIT/x86/x64] Unify and streamline emitter for operand-size override prefix (0x66)

Open

#11.788 aperta il 10 gen 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
arch-x64arch-x86area-CodeGen-coreclrhelp wanted

Metriche repository

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

Descrizione

Now, the emitter has a lot of code like below for adding operand-size override prefix (0x66) or estimating code size of it.

    if (attrSize == EA_2BYTE && ins != INS_movzx && ins != INS_movsx)
    {
        ...
    }

We need to refactor the emitter to unify and simplify the code.

cc @CarolEidt @tannergooding

category:implementation theme:emitter skill-level:beginner cost:small impact:small

Guida contributor

Direzione di ricerca
Esamina il codice dell'emettitore in RyuJIT che controlla attrSize == EA 2BYTE per le istruzioni tranne movzx/movsx e refactoring per semplificare creando un approccio unificato per aggiungere il prefisso 0x66.
Tech stack
cpp
Dominio
compilersbackend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80